Re: [MMUSIC] I-D Action: draft-ietf-mmusic-sctp-sdp-03.txt

["Horvath, Ernst" <[e-mail]>]

 
I am sorry to say that the treatment of attributes in this I-D is a bit of a mess. 

- In 4.3 the attributes are introduced with a colon-notation, e.g. "a=stream:", but the ABNF definitions in 5.2 through 5.7 use a notation "a=stream=" etc. The colon-notation is the more common one for SDP attributes.

- Looking at the examples, I suspect that 'sctpmap' is the only real attribute and the rest is meant to be parameters of an attribute 'webrtc-DataChannel', which appears in examples but is not defined in section 5. If this is correct then 'webrtc-DataChannel' needs to be added as a new attribute in section 5, and what is now in 5.2 - 5.7 should become sub-sections of this attribute definition and be renamed as "parameters".

- Section 11 says "This document defines two SDP session and media-level attributes", but lists only one (sctpmap). Was the missing one meant to be 'webrtc-DataChannel'?

Note: An SDP attribute 'label' is already defined (RFC 4574) and listed in the IANA registry. This would cause a name clash if 5.3 is really intended as attribute and not as parameter definition.

Thanks,
Ernst Horvath