Re: [MMUSIC] New draft version: draft-ietf-mmusic-data-channel-sdpneg-01

[Juergen Stoetzer-Bradler <[e-mail]>]

Paul,

Thank you for further comments.
Please see my remarks inserted below.

Thanks,
Juergen

On 09.03.2015 16:33, Paul Kyzivat wrote:
> Juergen,
>
> In general this is good. I have a few remaining comments:
>
> Section 5.2.2 says:
>
>    ... If an SDP offer contains both of
>    these parameters then such an SDP offer will be rejected.
>
> The use of "will" is confusing - it isn't normative. IMO it should either use "MUST" or else it 
> should say such usage is undefined.

[Juergen] The intention is indeed to request the answerer to reject sch an SDP offer.
Thus we could say:
"... If an SDP offer contains both of these parameters then such an SDP offer MUST be rejected."

>
> Then:
>
>    The SDP answer shall echo the same subprotocol, max-retr, max-time,
>    ordered parameters, if those were present in the offer, and may
>
> Again, "shall" is non-normative. IMO this should be SHALL or MUST.

[Juergen] Agreed. Would propose to say:
"The SDP answer SHALL echo the same subprotocol, max-retr, max-time, ordered parameters, ..."

>
> Then:
>
>    Data channel types defined in [I-D.ietf-rtcweb-data-protocol] are
>    mapped to SDP in the following manner:
>
>    DATA_CHANNEL_RELIABLE
>          a=dcmap:2 subprotocol="BFCP";label="channel 2"
>    ...
>
> "This is a bit unclear because these are *examples* using BFCP. (It also uses 'ordered=0' rather 
> than 'ordered=false'. I think it would be clearer as:
>
>    Data channel types defined in [I-D.ietf-rtcweb-data-protocol] are
>    mapped to SDP a=dcmap parameters in the following manner:
>
>    DATA_CHANNEL_RELIABLE
>          ordered=true
>
>    DATA_CHANNEL_RELIABLE_UNORDERED
>          ordered=false
>
>    DATA_CHANNEL_PARTIAL_RELIABLE_REXMIT
>          ordered=true;max-retr=NNN
>
>    DATA_CHANNEL_PARTIAL_RELIABLE_REXMIT_UNORDERED
>          ordered=false;max-retr=NNN
>
>    DATA_CHANNEL_PARTIAL_RELIABLE_TIMED
>          ordered=true;max-time=NNN
>
>    DATA_CHANNEL_PARTIAL_RELIABLE_TIMED_UNORDERED
>          ordered=false;max-time=NNN
>
> ('ordered=true' is default and may be omitted.)"

[Juergen] We fully agree an will update the text as you propose in next draft -02.

>
>     Thanks,
>     Paul
>
[snip]