Re: [rtcweb] [Tsv-art] Tsvart last call review of draft-ietf-rtcweb-ip-handling-11

You could just add “Unix” or “Unix-like” to before “connect”, or refer to these calls as “bind()” and “connect()”.

It is important that these are the semantics of Unix-like sockets; other APIs may not have a corresponding mechanism. Further, the UDP source address might change later (i.e., in TCP the “connect” always forces a static source IP address selected at the initiation of the TCP connection; in UDP, there’s no need for this to happen only once at the time the “connect()” is issued.

Joe

> On Feb 22, 2019, at 5:19 PM, Justin Uberti <juberti=40google.com@dmarc.ietf.org> wrote:
> 
> Thanks for your comments. See below.
> 
> On Sun, Feb 17, 2019 at 9:56 AM Joseph Touch <touch@strayalpha.com <mailto:touch@strayalpha.com>> wrote:
> Reviewer: Joseph Touch
> Review result: Ready
> 
> This document has been reviewed as part of the transport area review team's
> ongoing effort to review key IETF documents. These comments were written
> primarily for the transport area directors, but are copied to the document's
> authors and WG to allow them to address any issues raised and also to the IETF
> discussion list for information.
> 
> When done at the time of IETF Last Call, the authors should consider this
> review as part of the last-call comments they receive. Please always CC
> tsv-art@ietf.org <mailto:tsv-art@ietf.org> if you reply to or forward this review.
> 
> This document has no significant transport issues.
> 
> As a very minor issue, the document refers to the use of UDP "connect":
> 
>    Once a suitable remote IP has been determined, the implementation can
>    create a UDP socket, bind it to the appropriate wildcard address, and
>    tell it to connect to the remote IP.  Generally, this results in the
>    socket being assigned a local address based on the kernel routing
>    table, without sending any packets over the network.
> 
> It might be useful to be more clear that this is an OS command (not a protocol
> one). If the particular semantics of this command are relevant, that should be
> noted as well.
> 
> Agreed. Thoughts on what would make this clearer? 
> _______________________________________________
> Tsv-art mailing list
> Tsv-art@ietf.org
> https://www.ietf.org/mailman/listinfo/tsv-art