Accept any <proto> value

Author: JoTurk

unmarshal.go

@@ -874,31 +874,7 @@ func unmarshalMediaDescription(lex *lexer) (stateFn, error) { //nolint:cyclop
 		return nil, err
 	}
 
-	// Set according to currently registered with IANA
-	// https://tools.ietf.org/html/rfc4566#section-5.14
-	// https://tools.ietf.org/html/rfc4975#section-8.1
 	for _, proto := range strings.Split(field, "/") {
-		if !anyOf(
-			proto,
-			"UDP",
-			"RTP",
-			"AVP",
-			"SAVP",
-			"SAVPF",
-			"TLS",
-			"DTLS",
-			"SCTP",
-			"AVPF",
-			"TCP",
-			"MSRP",
-			"BFCP",
-			"UDT",
-			"IX",
-			"MRCPv2",
-			"FEC",
-		) {
-			return nil, fmt.Errorf("%w `%v`", errSDPInvalidNumericValue, field)
-		}
 		newMediaDesc.MediaName.Protos = append(newMediaDesc.MediaName.Protos, proto)
 	}
 

unmarshal_test.go

@@ -1812,19 +1812,6 @@ func TestUnmarshalMediaDescription_SetsPortRange(t *testing.T) {
 	}
 }
 
-func TestUnmarshalMediaDescription_Error_InvalidProto(t *testing.T) {
-	// proto token not in the allowed list
-	l := &lexer{
-		desc:      &SessionDescription{},
-		cache:     &unmarshalCache{},
-		baseLexer: baseLexer{value: "audio 9 WRONG/PROTO 0\r\n"},
-	}
-
-	st, err := unmarshalMediaDescription(l)
-	assert.Nil(t, st)
-	assert.ErrorIs(t, err, errSDPInvalidNumericValue)
-}
-
 func TestUnmarshalMediaTitle_Error_ReadLine(t *testing.T) {
 	// empty input
 	l := &lexer{