Re: [BEHAVE] WGLC: draft-ietf-behave-turn-tcp-04

[Simon Perreault <[e-mail]>]

Le 2009-08-07 13:00, Alfred E. Heggestad a écrit :
> 5.2. Receiving a Connect Request
>
> "Otherwise, the server MUST initiate an outgoing TCP connection. The
> local endpoint is the relayed transport address associated with the
> allocation. ..."
>
> ---> why does the local endpoint have to be the relayed transport address?
> is this a technical requirement to make the TURN/TCP protocol work?
> Normally an outgoing TCP connection using BSD sockets will use a
> random port as local endpoint. It is possible to use the same port
> as the TCP socket is listening on, but will require that things are
> initialized in the correct order. You should check that this does
> cause problems for implementors (ref: ICE tcp-SO discussion)

Here's one case where this is needed: when two TURN servers are talking 
to each other, the client needs to install a permission on each server 
for the remote socket. So each client needs to be know in advance where 
the connection is going to come from.

> 5.4. Receiving a ConnectionBind Request
>
> ---> 5th paragraph: "...Connect or Listen request..."
>
> I think the "Listen" request is not defined anymore ...

Right. Will be removed.

> - some of the text in the abstract is bold in the HTML version

Yes, this is because the HTML version is generated from the text version 
by heuristics. It got this part wrong.

> last line: "STUN Server" --> "TURN Server"

Right. Strange how this was never noticed before...

> 1. Introduction:
>
> "It also allows the client to initiate connections from that allocation
> to peers"
>
> would like to suggest instead:
>
> "It also allows the client to initiate outgoing TCP connections from
> that allocation to peers"

> ", and accept connection requests from peers made towards that allocation"
>
> would like to suggest instead:
>
> ", and accept incoming TCP connections from peers made towards that
> allocation"

Sure, why not.

Simon
-- 
DNS64 open-source   --> http://ecdysis.viagenie.ca
STUN/TURN server    --> http://numb.viagenie.ca
vCard 4.0           --> http://www.vcarddav.org