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

Mary Barnes <mary.ietf.barnes@gmail.com> Mon, 24 September 2012 21:14 UTC

Return-Path: <mary.ietf.barnes@gmail.com>
X-Original-To: bfcpbis@ietfa.amsl.com
Delivered-To: bfcpbis@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 52C3321F8820 for <bfcpbis@ietfa.amsl.com>; Mon, 24 Sep 2012 14:14:02 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -103.315
X-Spam-Level:
X-Spam-Status: No, score=-103.315 tagged_above=-999 required=5 tests=[AWL=0.283, BAYES_00=-2.599, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-1, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([64.170.98.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id SATlr19PZ5LW for <bfcpbis@ietfa.amsl.com>; Mon, 24 Sep 2012 14:14:01 -0700 (PDT)
Received: from mail-lb0-f172.google.com (mail-lb0-f172.google.com [209.85.217.172]) by ietfa.amsl.com (Postfix) with ESMTP id 00D5521F880C for <bfcpbis@ietf.org>; Mon, 24 Sep 2012 14:14:00 -0700 (PDT)
Received: by lbok13 with SMTP id k13so1679533lbo.31 for <bfcpbis@ietf.org>; Mon, 24 Sep 2012 14:14:00 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=roXXwDfQmlXb0+TLIZt/R+sDuXXVTUS2Aw+NSR8gQO4=; b=fuLd7zs7IttifO35r6crHDz6Zp0z3SgX71NF7AL42/y8OAKCbzmYDUN8ktXGxWT7UP u7ZOjqsw/wS0+0wnkqU2YJT5BJ0kM9oNaQmQq4ueggqvBdHCEmjdhyssHHBzUhYELKkH jH4kQVC8kQM0jB4vz7ECtz1HdVcx675gTEWhgENiBFTCw/qtTEMYnpie9RaQ986W499w UUTRIQdZjzvdU3ntFa4/i5/NMj9HD3f97ALJmVoJ7iE2LRo2PhC5wjIf6GEyTYMSN+gJ 9PRx+NPlerpZWBwTH7VYZkU6leJf0N87dUqNmyRQnFtaptpnSCRwmoKXpX45rK9oZ0bp TxxA==
MIME-Version: 1.0
Received: by 10.152.132.202 with SMTP id ow10mr11420648lab.51.1348521239825; Mon, 24 Sep 2012 14:13:59 -0700 (PDT)
Received: by 10.114.4.130 with HTTP; Mon, 24 Sep 2012 14:13:59 -0700 (PDT)
In-Reply-To: <92B7E61ADAC1BB4F941F943788C088280A5951@xmb-aln-x08.cisco.com>
References: <92B7E61ADAC1BB4F941F943788C088280A5951@xmb-aln-x08.cisco.com>
Date: Mon, 24 Sep 2012 16:13:59 -0500
Message-ID: <CAHBDyN6E+dAwx0tVJHWQvYPK=F+=ptq1O=wsH74ELxpNfCrYDw@mail.gmail.com>
From: Mary Barnes <mary.ietf.barnes@gmail.com>
To: "Charles Eckel (eckelcu)" <eckelcu@cisco.com>
Content-Type: multipart/alternative; boundary="f46d0430880a78cdf504ca791101"
Cc: "bfcpbis@ietf.org" <bfcpbis@ietf.org>
Subject: Re: [bfcpbis] WGLC for draft-ietf-bfcpbis-rfc4582bis
X-BeenThere: bfcpbis@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: BFCPBIS working group discussion list <bfcpbis.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/bfcpbis>, <mailto:bfcpbis-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/bfcpbis>
List-Post: <mailto:bfcpbis@ietf.org>
List-Help: <mailto:bfcpbis-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/bfcpbis>, <mailto:bfcpbis-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 24 Sep 2012 21:14:02 -0000

I have reviewed the document and it looks pretty much ready.

I do have a couple of questions for clarification:
- Section 5.1 - Payload Length: states that "receiving server MAY send an
Error message"
but I would think that the client should know about the error. Is there a
specific reason this is a MAY versus a SHOULD or even a MUST? Why would the
error message matter if it's not required?

- Section 5.2 - "M" after Table 2 - I have a similar question about the 'M'
bit. RFC 4582 states that message is rejected if there is an unrecognized M
bit, whereas this document states that an error message MAY be sent.
Certainly, 4582 lacked appropriate normative language, but if the error
message is just a MAY, then why bother?

There are also a few nits that should be fixed/clarified before progressing.

Section 5.2.6 - Note under table 5:    "intended being used" -> "intended
to be used"

Section 6.1.: "e.g.,"  was changed to "e.g.".   The former is correct.

Section 9.1: You lost some indented paragraphs in this section - i.e., they
got shifted to align with the other paragraphs in that section.  I'm not
sure if that was intentional.

Section 16:
- New Primitives.  The text says five were added, but only 4 are listed and
that's all that's all that's defined per Figure 1.
- New error codes.  The text says that 3 are aded, but it looks there's
really 5 that have been added.
- Typo.  Referring to section 12.4.2 states that the change was from
SUPPORTED-PRIMITIVES
to SUPPORTED-PRIMITVIES but the latter is really the typo.

Regards,
Mary

On Fri, Sep 21, 2012 at 7:36 PM, Charles Eckel (eckelcu)
<eckelcu@cisco.com>wrote:

> 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
>