#507 integer value parsing, Re: APPSDIR review of draft-ietf-httpbis-p5-range-24

Julian Reschke <julian.reschke@gmx.de> Thu, 07 November 2013 00:50 UTC

Return-Path: <julian.reschke@gmx.de>
X-Original-To: ietf@ietfa.amsl.com
Delivered-To: ietf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 09A5A21E818E for <ietf@ietfa.amsl.com>; Wed, 6 Nov 2013 16:50:56 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -104.31
X-Spam-Level:
X-Spam-Status: No, score=-104.31 tagged_above=-999 required=5 tests=[AWL=-1.711, BAYES_00=-2.599, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id em-aXXfqXc21 for <ietf@ietfa.amsl.com>; Wed, 6 Nov 2013 16:50:51 -0800 (PST)
Received: from mout.gmx.net (mout.gmx.net [212.227.17.21]) by ietfa.amsl.com (Postfix) with ESMTP id 0F0F111E8160 for <ietf@ietf.org>; Wed, 6 Nov 2013 16:50:51 -0800 (PST)
Received: from [31.133.162.78] ([31.133.162.78]) by mail.gmx.com (mrgmx002) with ESMTPSA (Nemesis) id 0MAPoQ-1VSe0p1gZG-00Be9F for <ietf@ietf.org>; Thu, 07 Nov 2013 01:50:50 +0100
Message-ID: <527AE3E8.3000207@gmx.de>
Date: Wed, 06 Nov 2013 16:50:48 -0800
From: Julian Reschke <julian.reschke@gmx.de>
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.1.0
MIME-Version: 1.0
To: S Moonesamy <sm+ietf@elandsys.com>, apps-discuss@ietf.org, draft-ietf-httpbis-p5-range.all@tools.ietf.org
Subject: #507 integer value parsing, Re: APPSDIR review of draft-ietf-httpbis-p5-range-24
References: <6.2.5.6.2.20131028101123.0e37a698@elandnews.com>
In-Reply-To: <6.2.5.6.2.20131028101123.0e37a698@elandnews.com>
Content-Type: text/plain; charset="ISO-8859-1"; format="flowed"
Content-Transfer-Encoding: 7bit
X-Provags-ID: V03:K0:m1CkxN9vijRQHyta2JY6hjp+uBXrDLP9fSAJFraa3SZvP1NMJMm F22T24mZX69Pna/rdoJjUxkyGA51O21rjh1yo20BlXHL4P5i9mPKG2XHwdPbdv6MpgUJsDu pIv+SOscndfem24KknOJA5glK+42Ifkth7qGX9UylWO+c5mJ/dsRAEZ+7hZIah8vGgDpw+n uUjUV4LFfldJP6E+A3VFQ==
Cc: iesg@ietf.org, ietf-http-wg@w3.org, ietf@ietf.org
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/ietf>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 07 Nov 2013 00:50:56 -0000

On 2013-10-29 01:13, S Moonesamy wrote:
> ...

I have been selected as the Applications Area Directorate reviewer for
> this draft (for background on APPSDIR, please see
> http://trac.tools.ietf.org/area/app/trac/wiki/ApplicationsAreaDirectorate ).
>
>
> Please resolve these comments along with any other Last Call comments
> you may receive. Please wait for direction from your document shepherd
> or AD before posting a new version of the draft.
>
> In Section 2.1:
>
>    "In the byte range syntax, first-byte-pos, last-byte-pos, and suffix-
>     length are expressed as decimal number of octets.  Since there is no
>     predefined limit to the length of a payload, recipients ought to
>     anticipate potentially large decimal numerals and prevent parsing
>     errors due to integer conversion overflows."
>
> There is a RFC 2119 "should" in Section 3.3.2 of
> draft-ietf-httpbis-p1-messaging-24 about integer conversion and a
> reference to Section 9.3 of that draft.  I may have missed integer
> conversation issues in the reviews.  I suggest doing a verification to
> verify that there is adequate text where it is applicable.
> ...

We discussed this during this weeks WG meeting, and the consensus was 
that we want to make both instances a "MUST".

The proposed change is here: 
<http://trac.tools.ietf.org/wg/httpbis/trac/attachment/ticket/507/507.diff>

Best regards, Julian