Hi Al, Thank You for the review! Please see inline. >Reviewer: Al Morton >Review result: Has Nits > >OPS-DIR review of MSRP over Data Channels >draft-ietf-mmusic-msrp-usage-data-channel-21 > > This document specifies how the Message Session Relay Protocol (MSRP) > can be transported as a WebRTC data channel sub-protocol, using the > SDP offer/answer generic data channel negotiation framework to > establish such a channel. > > Summary: Operational impact of this protocol appears to be minimal, if any. > > Gen-ART review has found some Nits that must be resolved. They were solved and implemented in the -22 version of the draft. >Key Notes from the Doc Shepherd's write-up [0]: > It seems that this document was revived after being abandoned, when 3GPP > noticed the change in status and recognized use of the draft as a normative > reference in 3GPP Specifications. > > There are no known implementations of the protocol, and thus no operational > issues at this time. However, some vendors (of???) have indicated plans to > implement. Based on my knowledge, vendors of both endpoints and network nodes (gateways etc). > Further investigation indicates that there are Normative References > to drafts that are more than a year old. For example, the 2 refs below: > ...the channel is > negotiated using the SDP-based external negotiation method defined in > [I-D.ietf-mmusic-data-channel-sdpneg]. > > The following WebRTC data channel property values > [I-D.ietf-rtcweb-data-channel] apply to an MSRP data channel: > >Also, from the Normative References section: > > [I-D.ietf-rtcweb-data-protocol] > Jesup, R., Loreto, S., and M. Tuexen, "WebRTC Data Channel > Establishment Protocol", draft-ietf-rtcweb-data- > protocol-09 (work in progress), January *2015*. > > [I-D.ietf-rtcweb-data-channel] > Jesup, R., Loreto, S., and M. Tuexen, "WebRTC Data > Channels", draft-ietf-rtcweb-data-channel-13 (work in > progress), January *2015*. All drafts mentioned above are part of cluster 238. Many drafts in the cluster are done, but the publications have been on hold for a while, as the idea is to publish them all at once. --- Nits: >2. Conventions > The key words "MUST", "MUST NOT", "REQUIRED", "SHALL", "SHALLNOT", s/"SHALLNOT"/"SHALL NOT"/ This has been fixed in version -22. Maybe you were looking at version -21? Regards, Christer [0] https://datatracker.ietf.org/doc/draft-ietf-mmusic-msrp-usage-data-channel/shepherdwriteup/ -- last-call mailing list last-call@xxxxxxxx https://www.ietf.org/mailman/listinfo/last-call
