[Last-Call] Genart last call review of draft-ietf-rtcweb-sdp-11

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Reviewer: Peter Yee
Review result: Ready with Nits

I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair.  Please treat these comments just
like any other last call comments.

For more information, please see the FAQ at

<https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.

Document: draft-ietf-rtcweb-sdp-11
Reviewer: Peter Yee
Review Date: 2020-04-28
IETF LC End Date: 2020-04-28
IESG Telechat date: Not scheduled for a telechat

Summary: This informative specification gives examples of using SDP to
negotiate session capabilities between WebRTC peers.  I lack the ability to
machine check each example for correctness, but I’ve read them and offered
suggestions to improve clarity and consistency.  I’ve also marked a large
number of nits that should be fixed prior to the document being submitted to
the RFC Editor to prevent gnashing of teeth and pulling of hair. ;-)  [Ready
with nits]

Major issues: None.

Minor issues:  None.

Nits:
(Given the difficulty in specifying a location within the examples, I’m relying
on the page numbers, section numbers, and other identifying marks to point out
where changes would be helpful.  That means you have to use the text version of
the draft to find what I'm specifying)

General:

Change “use-cases” to “use cases”.  The hyphen is unnecessary.  There’s also
one instance of “use-case” (singular) that needs to be changed.

Change “48khz” to “48 kHz” throughout the document.  That aligns with preferred
SI formatting and the proper name for the unit.

Strongly consider changing “Audio” to “audio”, “Video” to “video”, “Stream” to
“stream”, and “Session” to “session” throughout the document (with the obvious
exception of at the beginning of sentences and in section titles).  RFC 3550
does not capitalize these terms.

There’s inconsistent usage of “n-way” throughout the document.  To wit,
sometimes I see the number (generally n is 1 or 2) spelled out in English,
other times it is shown as numeric digits.  Sometimes the hyphen is present,
sometimes it is missing.  “Way” is sometimes capitalized, sometimes it’s not. 
I don’t care whether you usage the English word for a number or the actual
digits, but choose one for and use it consistently.  The hyphen is not
optional.  “Way” is lower case in this usage.  There are way too many of these
occurrences for me to mark them individually, but you should be able to find
them with a different searches.

Change “reduced size” to “reduced-size” throughout the draft.  This change
aligns the draft with RFC 5506.

In the “RFC#/Notes” column in several example, when RFC 5104 FIR is explained,
drop the word “Frame” from the entry.  RFC 5104 does not include “Frame” in the
term.  FIR is just “Full Intra Request”.

Change “sreams” to “streams” everywhere.  Change “stram” to “stream” everywhere.

Consider changing “Offer” to “offer”, “Answer” to “answer”, “Offerer” to
“offerer”, and “Answerer” to “answerer” to align with RFC 3264.

Change “end-point” to “endpoint” everywhere.

Specific:

Page 1, Abstract, 1st paragraph, 1st sentence: append a comma after “video”.

Page 1, Abstract, 1st paragraph, 2nd sentence: change “With in” to “Within”. 
Change “Rtcweb” to “WebRTC”.  Change “protocol” to “Protocol”.

Page 1, Abstract, 2nd paragraph, change “Rtcweb” to “WebRTC”, to maintain
alignment with draft-ietf-rtcweb-overview usage.

Page 3, section 1, 1st paragraph, 1st sentence: change “Javascript” to
“JavaScript”.  Append a comma after “updating”.

Page 3, section 3, title: delete “the”.

Page 3, section 3, 1st paragraph, 2nd sentence: delete “the”.

Page 3, 2nd paragraph, 2nd sentence: insert “a” before “multimedia”.  Append a
comma after “addresses”.  Append a comma after “invitation”.

Page 3, section 3, 3rd paragraph: append a comma after “[RFC3550]”.  Move
“[RFC7826]” between “Protocol” and “applications”.

Page 4, 1st paragraph: Insert “The figure” at the beginning of the sentence. 
Replace “Below figure” with “below”.  Insert “a” before “high-level”.

Page 5, 1st paragraph (after the figure), 1st sentence: insert “using a” before
“JavaScript”.

Page 5, 2nd numbered item: insert “a” before “participant’s”.

Page 5, section 4, 1st paragraph, 1st sentence: insert “the” before “SDP”. 
Append a comma after “updating”.

Page 5, section 4, 1st paragraph, 1st sentence: insert “to” before “[RFC3264]”.
 Change “operation” to “operations”.

Page 6, 1st paragraph, 1st sentence: change “rule” to “rules”.

Page 6, 1st paragraph, 2nd sentence: insert “a” before “protocol”.  Delete
“with”.  Insert “the” before “participants”.  Change “from” to “with”.  Change
“others” to “other”.  Delete “perspective”.  Append “messages” (or another
similar word) after “SDP”.

Page 6, 2nd paragraph, 1st sentence: insert a space before “the protocol”.

Page 6, 4th paragraph, 1st sentence: change “With in” to “Within”.  Change
“them” to “themselves”.

Page 6, 4th paragraph, 2nd sentence: Insert “The” before “JSEP”.  Insert “and”
before “transport”.

Page 6, section 5, 1st paragraph: change “web based” to “web-based”.  Change
“below” to “follows”.

Page 6, 1st bullet item: change “Audio only” to “audio-only”.  Change “Video
only” to “video-only”.  Change “Audio/Video” to “audio/video”.  Change
“Sessions” to “sessions”.  RFC 3550 does not capitalize these words.

Page 6, 4th bullet item: insert “the” before “ICE”.

Page 6, 5th bullet item: change “RTCP based” to “RTCP-based”.

Page 6, 6th bullet item: insert “or” before “dual”.  Change “dual-stack based”
to “dual stack-based”.

Page 7, 1st bullet item: change “BUNDLE based” to “BUNDLE-based”.

Page 7, section 5.1, 2nd bullet item: change “apriori” to “a priori”.  Insert
“a” before “successful”.

Page 7, section 5.1, 3rd bullet item: change “is” to “are”.  Append a comma
after “detailed”.

Page 7, section 5.1, 4th bullet item: change “Eventhough” to “Even though”.

Page 7, section 5.1, 5th bullet, 3rd sub-bullet: change the “an” before “ASCII”
to “a” in both places.  Change “ASCII” to “US-ASCII” in both occurrences to
align with IANA’s preferred usage.

Page 8, 1st bullet item: insert “The” at the beginning of the sentence:  Change
“Following” to “following”.

Page 8, 1st paragraph following the sub-bullets: insert “the” before
“[I-D.ietf-rtcweb-jsep]”.

Page 8, 2nd bullet item: change “a” to “an” before “RTP Session”.  Change the
“a” before “RTP Stream” to “an”.  Also review the general nit regarding
capitalization of terms.

Page 8, 3rd bullet item: append a comma after “H.264”.

Page 8, 4th bullet item: change “is” to “are”.

Page 8, 5th bullet item: change “is” to “are”.

Page 8, 6th bullet item: append “’s” after “Bob”.

Page 8, 7th bullet item: change “Mediastream” to “MediaStream”.  Insert “and”
before “MediaStreamTrack”.  Change “Ids” to “IDs”.  Insert “than” before
“and/or” append “in comparison to” after “random”.  Delete “than” before “the
ones”.

Page 9, 3rd bullet item: delete “section”.

Page 9, section 5.2.1, 1st paragraph: insert “the” before “SDP”.  Insert “a”
before “secure”.

Page 9, ladder diagram: see general nit “n-way”.

Page 14, section 5.2.2, ladder diagram heading: change the comma to a slash.

Page 14, section 5.2.2.1, 1st sentence: change “IPv4 only” to “IPv4-only”.

Page 19, RFC#/Notes column, RFC4585: change “NACK based” to “NACK-based”.

Page 19, RFC#/Notes column, RFC5104: delete an extraneous space after they
hyphen.

Page 24, RFC#/Notes column, RFC4585: change “NACK based” to “NACK-based”.

Page 24, section 5.2.3, title: change “Data Only” to “Data-only”.

Page 24, section 5.2.3, 1st sentence: change “setup” to “set up”.  Change
“file-transfer” to “file transfer”.  Change the comma after “file-transfer” to
“ or”.  Append a comma after “control”.

Page 25, ladder diagram: change “SCTP based” to “SCTP-based”.  Change
“DataChannel” to “data channel”.

Page 31, section 5.2.5, 2nd sentence: insert “the” before “first”.

Page 35, section 5.2.6: I had some difficulty in rewriting this sentence to
make sure it’s clear that no one mistakenly believes the audio is somehow muxed
into the video stream.  I ended up with “In this scenario Alice and Bob engage
in a one-way audio and video session with Bob receiving Alice's her
presentation slides as a video stream and her audio.”  I’m not entirely happy
with that either.  If you believe that the original wording is unlikely to be
mistaken in its intent by the reader, feel free to ignore this suggestion.

Page 36, ladder diagram: change “send only” to “send-only”.

Page 37, RFC#/Notes column, RFC3264: change “Send only” to “Send-only”

Page 38, RFC#/Notes column, RFC3264: change “Receive only” to “Receive-only”.

Page 39, section 5.2.7: insert “the” before “following”.  Change “setup” to
“set up”.  Append a period at the end of the sentence.

Page 40, 1st bullet item: delete “the” before “Alice”.

Page 40, 2nd bullet item: change “offerer’s” to “Offerer’s” to be consistent
with the previous bullet.  Or see the general nit concerning the terms “Offer”,
“Offerer”, “Answer”, and “Answerer” for a more general solution.  If accepting
that general nit, then a correction in the previous bullet item is needed and
this bullet item does not need to be changed.

Page 40, 1st paragraph after bullet items: insert “a” before “single”.  Insert
“of” before “the Media Streams”.  Consider making “Media Streams” lower case.

Page 40, ladder diagram, title: change the comma to “ and “.

Page 40, ladder diagram: change “address” to “addresses”.

Page 45, section 5.2.8, title: insert a space before “Video”.  Append a comma
after “Video”.

Page 45, ladder diagram, title: insert “ and “ before Data.

Page 45, ladder diagram: insert “an “ before “identical”.  Insert a space
before “Video” at the bottom of the diagram.

Page 50, ladder diagram, title: change the comma to “ and “.

Page 54, section 5.2.10, title: drop the parentheses.

Page 54, section 5.2.10, 1st sentence: change “show-cases” to “showcases”. 
Append a comma after “Video”.  Insert “the” before “data”.

Page 54, section 5.2.10, 2nd (partial) sentence: insert “a” before “unique”.

Page 55, 1st partial paragraph: insert “the” before “data”.

Page 59, section 5.2.11, 2nd sentence: insert “The” at the beginning of the
sentence.  Change “First” to “first”.  Change “setup” to “set up”.  Insert “an”
before “Audio”.  Insert “a” before “video”.

Page 60, ladder diagram, title: delete the space after “Only”.

Page 66, section 5.3, 1st sentence: change “coding, simulcast” to “coding and
simulcast”.

Page 66, section 5.3, 2nd sentence: insert “a” before “few”.

Page 66, section 5.3, 3rd sentence: change “to” to “of”.

Page 66, section 5.3.1, title: change “Sendonly” to “Send-only”.  Capitalize
“cameras”, “encodings”, and “camera” to match your other capitalization scheme
for most other section titles.

Page 66, section 5.3.1, 1st paragraph, 1st sentence: append “example” after
“SDP”.  Insert “an” before “Offer/Answer”.

Page 66, section 5.3.1, 1st paragraph, 2nd sentence: change “source” to
“sources”.

Page 66, section 5.3.1, 3rd, 4th, and 5th paragraphs: insert “The” at the
beginning of the paragraph.

Page 66, section 5.3.1, 5th paragraph: append “solely” after “sources”.

Page 67, 1st paragraph: Insert “The” at the beginning of the sentence:  delete
“the” before “5”.

Page 67, 2nd paragraph: delete “a” before “non-interactive”.

Page 67, ladder diagram: change “sendonly” to “send-only”.  Append a comma
after “H.264”.

Page 70, RFC#/Notes column, RFC3264: change “Send only” to “Send-only”.

Page 74, section 5.3.2, 1st paragraph, 2nd sentence: insert “is” before
“being”.  Change “stream” to “streams”.

Page 74, ladder diagram: insert “a” before “sendonly”.  Change “sendonly” to
“send-only”.  This section says the session has audio and a single video
source, but the diagram doesn’t show an audio offer being made.  Is that
correct?  Change “Codec” to “codec”.

Page 78, 1st paragraph: change “[RFC4588] style” to “[RFC4588]-style”.  Delete
the hyphen in “re-transmission” to align with RFC 4588 usage.

Page 78, 2nd and 3rd paragraphs: insert “The” at the beginning of the paragraph.

Page 79, ladder diagram: insert a space before “all muxed”.

Page 83, section 5.3.4, 2nd paragraph: change “Stream” to “Streams” and append
a comma.

Page 84 ladder diagram: change “audio,video” to “audio and video” at the bottom.

Page 88, section 5.3.5, 1st paragraph: insert “a” before “Simulcast”.  Change
“Simulcast” to “simulcast”.  Delete an extraneous “and”.  Change “[RFC5956]
style” to “[RFC5956]-style”.

Page 88, section 5.3.5, 2nd paragraph: append a comma after “mechanism”. 
Insert “with” before “one”.  Append a comma after “video streams”.  Append a
comma after “FEC streams”.  Insert “which” before “are sent”.

Page 89, ladder diagram: at the bottom of the diagram, you refer to this as “4
video streams”.  In the description on page 88, you referred to this as “2
simulcast video streams, and 2 associated FEC streams”.  Which one is it?  Are
FEC streams considered video streams?  Also, change the comma to “ and “. 
Change “Simulcast” to “simulcast”.  Append a comma after the “)”.

Page 93, section 5.4:  change “the” to “this”.  Change “exchange” to
“exchanges”.  Change “extension” to “extensions”.  Append a comma after
“scenarios”.

Page 93, section 5.4.1: change “audio-level” to “audio level”.

Page 97, section 5.4.2: change the sentence to start “This example of SDP
highlights the RTP header”

Page 97, ladder diagram: change “interest to audio” to “interest in audio”. 
Change “audio-levels” to “audio levels”.  Change “Audio-levels” to “Audio
levels”.  Change “CSRCS is” to “CSRCS are”.

Page 100, section 5.4.3, title: change “legacy” to “Legacy”.  Change “bundle”
to “Bundle”.

Page 100, section 5.4.3, 1st paragraph, 1st sentence: append a comma after
“WebRTC endpoint”.

Page 100, section 5.4.3, 1st paragraph, 2nd sentence: insert “an” before
“audio”.  Change “audio only” to “audio-only”.  Insert “the” before
“bundle-only”.

Page 100, section 5.4.3, 1st paragraph, last word: change the comma to a colon.

Page 100, section 5.4.3, 1st bullet item: Insert “The” at the beginning of the
sentence.  General nit regarding capitalization applies. Page 100, section
5.4.3, 2nd bullet: insert “the” before “BUNDLE”.

Page 100, NOTE: insert “the” before “BUNDLE”.

Page 101, ladder diagram: insert “the” between “accepts” and “audio” on the
side.

Page 105, section 5.4.4, 1st paragraph: change “which” to “that”.  Insert “an”
before “[RFC3264]”.  Change “RTP/AVP based” to “RTP/AVP-based”.  Insert “a”
before “DTLS”.

Page 105, section 5.4.4, 2nd paragraph: append a comma after “Bob”.  Append a
comma after “end-point” (noting general nit about end-point).  Insert “the”
before “procedures”.  Insert “a” before “DTLS”.  Change “DTLS-SRTP based” to
“DTLS-SRTP-based”.

Page 106, ladder diagram: change “End-point” to “endpoint” (lowercase, no
hyphen) twice.  Change “Legacy compliant” to “Legacy-compliant”.  Change “dtls”
to “DTLS”.  Change “rtcp” to “RTCP”.  On the side, change “legacy compliant” to
“legacy-compliant”.

Page 108, RFC#/Notes column, RFC4585, change “loss” to “Loss”.

Page 108, RFC#/Notes column, RFC4566 change “RTP/AVP based” to “RTP/AVP-based”
in two places.

Page 109, section 7, 2nd paragraph: append a comma after “TLS transport”.

Page 109, section 8, 2nd sentence: insert “a” before “syntax”.

Page 118, section A.1, 2nd sentence: change “compliant to” to “compliant with”.

Page 118, 2nd bullet item: insert “a” before “64”.  Change “64 bit” to
“64-bit”.  Append a comma after “’addrtype’”.

Page 118, 3rd bullet item: change “cases” to “case”.

Page 119, 1st bullet item: change “with” to “and have”.

Page 119, 3rd bullet item: change “WEBRTC” to “WebRTC”.

Page 119, section A.1.2: insert “The” at the beginning of the sentence.  Change
“Following” to “following”.

Page 119, section A.1.2, 1st bullet item: insert “the” before “dummy”.

Page 119, section A.1.2, 3rd bullet item: insert “The” at the beginning of the
sentence.

Page 119, section A.1.2, 5th bullet item: delete “in” before “included”.

Page 119, section A.1.2, 7th bullet item: append a comma after “retransmission”.

Page 119, section A.1.2, 8th bullet item: append a period at the end of the
sentence.

Page 119, section A.1.2, 10th bullet item: change “i.e” to “i.e.,”, i.e., add a
period and a comma.

Page 120, 4th bullet item: change “lines” to “line”.

Page 120, 6th bullet item: insert “the” before “following”.

Page 120, 8th bullet item: change “session level” to “session-level”.  Change
“lip same” to “same lip”.

Page 121, section A.1.3, title: change “DataChannel” to “Data Channel”.

Page 121, section A.1.3, change “datachannel” to “data channel”.

Page 121, 2nd bullet item: change “An JSEP” to “A JSEP”.

Page 121, 5th bullet item: insert “an” before “SCTP”.



-- 
last-call mailing list
last-call@xxxxxxxx
https://www.ietf.org/mailman/listinfo/last-call




[Index of Archives]     [IETF Annoucements]     [IETF]     [IP Storage]     [Yosemite News]     [Linux SCTP]     [Linux Newbies]     [Mhonarc]     [Fedora Users]

  Powered by Linux