search

rtcweb-wg / rtcweb-transport

draft-ietf-rtcweb-transport
2 stars 2 forks source link

Editorial from Ben Campbell #29

Closed alvestrand closed 7 years ago

alvestrand commented 7 years ago

From Ben Campbell's review:

Editorial Comments:

  • Please expand TURN, SSL, and ICE on first mention.
  • 3.4, 5th paragraph from end: "using TCP only between the endpoint and its relay" I think this means the use of TCP on the hop between the endpoint and the relay and not on other hops, but some may read it as the use of TCP but not other protocols on that hop.
  • 4.2: There are a few instances of text of the form of "... packets...use a single DSCP code point", which I think would be more clear with s/a single/the same.
alvestrand commented 7 years ago

Second point is fixed by #31 . I'm not sure the third point is an improvement.

alvestrand commented 7 years ago

SSL is used exactly once, as part of the acronym "TURN/SSL". This is also the first mention of TURN, but the second was much more convenient to attach the expansion to.

Given that it was so awkward to attach them, I included a new section listing the "used protocols" instead.