Re: [bfcpbis] WGLC for draft-ietf-bfcpbis-rfc4582bis

(as an individual)

I have reviewed the draft and I think that overall it is in very good shape. The only comment I would consider major is one dealing with section 4's description of server-initiated transactions:

Section 4 reads as follows:

   There are two types of transaction in BFCP: client-initiated
   transactions and server-initiated transactions.  Client-initiated
   transactions consist of a message from a client to the floor control
   server and a response from the floor control server to the client.
   Correspondingly, server-initiated transactions consist of a message
   from the floor control server to a client and the associated
   acknowledgement message from the client to the floor control server.
   Both messages can be related because they carry the same Transaction
   ID value in their common headers.

This is accurate when using UDP as the transport, but does not account for the case of using TCP as the transport. A more detailed and accurate description is provided in Section 8, which reads as follows:

   In BFCP, there are two types of transactions: client-initiated
   transactions and server-initiated transactions (notifications).
   Client-initiated transactions consist of a request from a client to a
   floor control server and a response from the floor control server to
   the client.  The request carries a Transaction ID in its common
   header, which the floor control server copies into the response.
   Clients use Transaction ID values to match responses with previously
   issued requests.

   Server-initiated transactions consist of a single message from a
   floor control server to a client.  Since they do not trigger any
   response, their Transaction ID is set to 0 when used over reliable
   transports, but must be non-zero and unique in the context of
   outstanding transactions over unreliable transports.

I suggest replacing the section 4 text with the following introductory level text, that currently exists in section 8 .

   In BFCP, there are two types of transactions: client-initiated
   transactions and server-initiated transactions (notifications).

The rest of the details currently in section 4 can  replaced with a reference to section 8.

I also have the following minor  comments.

1) Section 5.2.6, note after table 5
r/is intended being used/is intended to be used

2) Section 5.3.14 current reads as follows:

   Floor participants and chairs acknowledge the receipt of a subsequent
   FloorRequestStatus message from the floor control server when
   communicating over unreliable transport.

I suggest the following rewording:

   When communicating over unreliable transport, floor participants and chairs 
   acknowledge the receipt of a subsequent FloorRequestStatus message from 
   the floor control server by sending an FloorRequestStatusAck.

3) Similar change for section 5.3.15.

4) Section 6.2, currently contains the following fragment:

   Only allowing exactly one BFCP message or message segment per UDP
   datagram.  

This is covered previously in the paragraph, so this fragment can be removed.

5) Section 6.2,
r/Entities MUST only have at most one/Entities MUST have at most one

6) Section 6.2.1
r/attempt, this is identical/attempt. This is identical

Cheers,
Charles

> -----Original Message-----
> From: bfcpbis-bounces@ietf.org [mailto:bfcpbis-bounces@ietf.org] On
> Behalf Of Charles Eckel (eckelcu)
> Sent: Friday, September 21, 2012 5:36 PM
> To: bfcpbis@ietf.org
> Subject: Re: [bfcpbis] WGLC for draft-ietf-bfcpbis-rfc4582bis
> 
> Just a gentle reminder that we are into the last week for comments. Please
> review the draft and submit your comments by the Sept 28th, 2012 deadline
> (not 2011 as erroneously typed previously)
> 
> Thanks,
> Charles (as co-chair)
> 
> > -----Original Message-----
> > From: Charles Eckel (eckelcu)
> > Sent: Friday, September 07, 2012 7:06 AM
> > To: bfcpbis@ietf.org
> > Subject: WGLC for draft-ietf-bfcpbis-rfc4582bis
> >
> > (As WG co-chair)
> >
> > This is to announce a working group last call for draft-ietf-bfcpbis-
> > rfc4582bis, "The Binary Floor Control Protocol (BFCP)".
> > http://datatracker.ietf.org/doc/draft-ietf-bfcpbis-rfc4582bis/
> >
> > This is intended as a Standards Track RFC, obsoleting RFC 4582.
> > Please respond to the list by September 28th 2011 (i.e. 3 weeks) with any
> > comments.
> >
> > It is helpful to attempt to categorize your comment (e.g. technical issue vs.
> > editorial), and also  to provide any replacement text you feel is necessary.
> > If you review the document and have no comments, please tell the chairs
> > that you have reviewed it. This is always useful information in assessing
> the
> > degree of WG review and consensus behind the document.
> > Note, another WGLC for draft-ietf-bfcpbis-rfc4583bis will be run in
> parallel.
> >
> > Cheers,
> > Charles
> _______________________________________________
> bfcpbis mailing list
> bfcpbis@ietf.org
> https://www.ietf.org/mailman/listinfo/bfcpbis