Re: [bfcpbis] I-D Action: draft-ietf-bfcpbis-rfc4582bis-07.txt

Hi Tom,

I had a close look at the changes, and overall they look okay to me.
During my review, I did notice a few editorial things that are worth fixing:
1) many of the references to [16] do not work correctly, including the first reference in section 3.2, as well as those in section 6.2.
2) For the note added in section 5.1, you have an extra 'a' and are missing a comma:
r/BFCP is designed to a achieve small message size, as explained in Section 1 and BFCP entities/BFCP is designed to achieve small message size, as explained in Section 1, and BFCP entities
3) Section 6.2:
r/ security considerations in Section 6 also applies/security considerations in Section 6 also apply
4) section 6.2.3:
r/A Denial of Service (DoS) attacks/A Denial of Service (DoS) attack
5) Section 8, I think we should rework the following paragraph:

   When using BFCP over unreliable transport, it is also required to
   choose values that let the receiver distinguish the reception of the
   next message in a sequence of BFCP messages from a retransmission of
   a previous message.  Therefore, BFCP entities using unreliable
   transport SHOULD use monotonically increasing values for the
   Transaction ID.

To read as follows:

   When using BFCP over an unreliable transport, it is important that the initiator 
   of a transaction choose a Transaction ID value that lets the receiver distinguish
   the reception of the next message in a sequence of BFCP messages from a retransmission of
   a previous message.  Therefore, BFCP entities using an unreliable  transport SHOULD use
   monotonically increasing Transaction ID values.

6) Section 8, 
r/When using BFCP over unreliable transports/When using BFCP over an unreliable transport
Note, a similar change was made in many places in order to address comments from Gonzalo; however, a global search and replace should be done to catch numerous other places in which it still occurs.

Cheers,
Charles

> -----Original Message-----
> From: bfcpbis-bounces@ietf.org [mailto:bfcpbis-bounces@ietf.org] On
> Behalf Of Charles Eckel (eckelcu)
> Sent: Wednesday, December 19, 2012 2:08 PM
> To: Tom Kristensen (tomkrist); bfcpbis@ietf.org
> Cc: 'Tom Kristensen'
> Subject: Re: [bfcpbis] I-D Action: draft-ietf-bfcpbis-rfc4582bis-07.txt
> 
> Everyone who submitted WGLC comments, please confirm this version
> addresses all your comments satisfactorily.
> Everyone else, please review the entire draft carefully and share any
> comment or concerns you have.
> Thanks to Tom for incorporating all the feedback and posting this update.
> 
> Happy Holidays!
> Charles
> 
> > -----Original Message-----
> > From: bfcpbis-bounces@ietf.org [mailto:bfcpbis-bounces@ietf.org] On
> > Behalf Of Tom Kristensen (tomkrist)
> > Sent: Wednesday, December 19, 2012 5:45 AM
> > To: bfcpbis@ietf.org
> > Cc: 'Tom Kristensen'
> > Subject: Re: [bfcpbis] I-D Action: draft-ietf-bfcpbis-rfc4582bis-07.txt
> >
> > On 12/19/2012 02:20 PM, internet-drafts@ietf.org wrote:
> > > A New Internet-Draft is available from the on-line Internet-Drafts
> > directories.
> > >   This draft is a work item of the Binary Floor Control Protocol Bis
> Working
> > Group of the IETF.
> > >
> > > 	Title           : The Binary Floor Control Protocol (BFCP)
> > > 	Author(s)       : Gonzalo Camarillo
> > >                            Keith Drage
> > >                            Tom Kristensen
> > >                            Joerg Ott
> > >                            Charles Eckel
> > > 	Filename        : draft-ietf-bfcpbis-rfc4582bis-07.txt
> > > 	Pages           : 89
> > > 	Date            : 2012-12-19
> > >
> >
> > Changes from -06:
> > - Language clarification - mail on list from Charles Eckel 2012-10-13
> > - Language clarification - use "BFCP connection" across the document,
> >    change three occurences of "BFCP association".
> > - Resolved a lot of issues, small and large - mail on list 2012-10-24
> >    after WGLC from Gonzalo Camarillo. Also, later mail discussion
> >    with regard to 5 of the issues.
> > - Added reference to RFC 5763, Section 6.7 - mail on list from Christer
> >    Holmberg 2012-11-28.
> >
> >
> > To my knowledge, in this draft version all the issues from WGLC and
> > after are answered and fixed if appropriate.
> >
> >
> > -- Tom
> > _______________________________________________
> > bfcpbis mailing list
> > bfcpbis@ietf.org
> > https://www.ietf.org/mailman/listinfo/bfcpbis
> _______________________________________________
> bfcpbis mailing list
> bfcpbis@ietf.org
> https://www.ietf.org/mailman/listinfo/bfcpbis