Merge pull request #53 from muzzammilshahid/clean-encode-cbor

Don't return serializer type in EncodeCBOR

serializers/cbor.go

@@ -9,6 +9,8 @@ import (
 	"github.com/xconnio/wampproto-go/messages"
 )
 
+const CBORSerializerID = 3
+
 var cborEncoder cbor.DecMode //nolint:gochecknoglobals
 
 type CBORSerializer struct{}
@@ -44,16 +46,16 @@ func (c *CBORSerializer) Static() bool {
 	return false
 }
 
-func EncodeCBOR(args []any, kwargs map[string]any) ([]byte, int, error) {
+func EncodeCBOR(args []any, kwargs map[string]any) ([]byte, error) {
 	var payload []any
 	payload = append(payload, args)
 	payload = append(payload, kwargs)
 	payloadData, err := cbor.Marshal(payload)
 	if err != nil {
-		return nil, 0, err
+		return nil, err
 	}
 
-	return payloadData, CBORSerializerValue, nil
+	return payloadData, nil
 }
 
 func DecodeCBOR(data []byte) ([]any, map[string]any, error) {

serializers/helpers.go

@@ -6,13 +6,6 @@ import (
 	"github.com/xconnio/wampproto-go/messages"
 )
 
-const (
-	JSONSerializerValue     = 1
-	MsgPackSerializerValue  = 2
-	CBORSerializerValue     = 3
-	ProtobufSerializerValue = 16
-)
-
 func ToMessage(wampMsg []any) (messages.Message, error) {
 	messageType, _ := messages.AsInt64(wampMsg[0])
 	var msg messages.Message

serializers/json.go

@@ -6,6 +6,8 @@ import (
 	"github.com/xconnio/wampproto-go/messages"
 )
 
+const JSONSerializerID = 1
+
 type JSONSerializer struct{}
 
 func (j *JSONSerializer) Serialize(message messages.Message) ([]byte, error) {

serializers/msgpack.go

@@ -6,6 +6,8 @@ import (
 	"github.com/xconnio/wampproto-go/messages"
 )
 
+const MsgPackSerializerID = 2
+
 type MsgPackSerializer struct{}
 
 func (m *MsgPackSerializer) Serialize(message messages.Message) ([]byte, error) {