Re: [bfcpbis] Comments on draft-ietf-bfcpbis-rfc4583bis-03

[Tom Kristensen <[e-mail]>]

On 10/24/2012 04:27 PM, Gonzalo Camarillo wrote:
> Folks,
>    
[...]
> Comments on draft-ietf-bfcpbis-rfc4583bis-03
>
> Section 3 includes a discussion about how to set the port field. That
> discussion is only relevant to TCP. The new draft needs to explain that
> and add a discussion about port handling in UDP.
>    

Good catch. Reorganizing the text and adding this for UDP:

   "When UDP is used as transport, the port field contains the
    port to which the remote endpoint will direct BFCP messages
    regardless of the value of the 'setup' attribute."

> Also, the document needs to discuss what is the equivalent of
> establishing a TCP connection (i.e., it allows endpoints to start
> exchanging BFCP messages) in UDP.
>    

The term "BFCP connection" is used in rfc4582bis/rfc4583bis independent 
of underlying transport.

  (For rfc4582bis: I propose we keep this common term regardless of 
underlying transport and change the three occurrences of "BFCP 
association" in Section 6.2 and 8.31 to "BFCP connection" as well.)

However, we do indeed need to specify the counterpart of Section 7 "TCP 
Connection Management" for UDP as transport. Will add a sentence or two, 
since using UDP as transport is quite straight forward. Will also need 
to add a UDP description to Section 8, i.e. mandate using the 'setup' 
attribute when DTLS is used.

Added to start of Section 7, now renamed to "BFCP Connection Management":
   "BFCP connections may use TCP or UDP as underlying transport. BFCP
    entities exchanging BFCP messages over UDP will direct the BFCP
    messages to the peer side connection address and port provided in
    the SDP 'm' line. TCP connection management is more complicated
    and is described below."
And the subsection named "TCP Connection Management" follows.

Added this sentence at the end of Section 8:
   "Endpoints that use the offer/answer model to establish a DTLS 
association MUST
    support the 'setup' attribute, as defined in RFC 4145. When
    DTLS is used with UDP, the 'setup' attribute indicates which of the 
endpoints
    (client or floor control server) initiates the DTLS association setup."

> Section 6 contains the following new paragraph:
>
> " Note: In [15] 'm-stream' was erroneously used in Section 9.  Although
>    the example was non-normative, it is implemented by some vendors.
>    Therefore, it is RECOMMENDED to support parsing and interpreting
>    'm-stream' the same way as 'mstrm' when receiving."
>
> The text should clarify (or be more explicit about) whether existing
> implementations are floor control server implementations or client
> implementations. The idea is that new implementers know clearly what
> exactly they need to support in order to be backwards compatible with
> those legacy implementations (whose implementers did not read RFCs but
> only the examples :-) ).
>    

Yeah, what kind of developers do this kind of things? :-P

Usage of a=floorid (and mstrm/m-stream) applies to endpoints willing to 
act as server, will add this to the second sentence in the note:
   "[...] some vendors and occurs in cases where the endpoint is willing 
to act as an server."

> The last paragraph of Section 8 discusses which entity behaves as the
> TLS server. Do we need a similar discussion for DTLS?
>    

Indeed. Handled above.

-- Tom