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

Julian Reschke <julian.reschke@gmx.de> Thu, 07 November 2013 15:30 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 4E05821E821E for <ietf@ietfa.amsl.com>; Thu, 7 Nov 2013 07:30:21 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -104.293
X-Spam-Level:
X-Spam-Status: No, score=-104.293 tagged_above=-999 required=5 tests=[AWL=-1.694, 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 XM2CUwp4yMPU for <ietf@ietfa.amsl.com>; Thu, 7 Nov 2013 07:30:16 -0800 (PST)
Received: from mout.gmx.net (mout.gmx.net [212.227.17.22]) by ietfa.amsl.com (Postfix) with ESMTP id 1198C21E80DD for <ietf@ietf.org>; Thu, 7 Nov 2013 07:30:12 -0800 (PST)
Received: from [31.133.151.131] ([31.133.151.131]) by mail.gmx.com (mrgmx003) with ESMTPSA (Nemesis) id 0LvEZe-1VmaX22UAE-010MnJ for <ietf@ietf.org>; Thu, 07 Nov 2013 16:30:11 +0100
Message-ID: <527BB202.4080103@gmx.de>
Date: Thu, 07 Nov 2013 07:30:10 -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: Re: #507 integer value parsing, Re: APPSDIR review of draft-ietf-httpbis-p5-range-24
References: <6.2.5.6.2.20131028101123.0e37a698@elandnews.com> <527AE3E8.3000207@gmx.de>
In-Reply-To: <527AE3E8.3000207@gmx.de>
Content-Type: text/plain; charset="ISO-8859-1"; format="flowed"
Content-Transfer-Encoding: 7bit
X-Provags-ID: V03:K0:WY/1TdlyQIVIapI2OzJVNoHRqLrCpOzcS4Qwnd9rZqAyMJzFoEO cwxePAHyLchvwICKCuGzALG37c+F53U9uCwmv44AeqA1BkzOaqyOsNVgb6WXZCLGhX/mZ7k 37wqB9ySn7UgMbuIDJDCPMcFWYEYazdN0AjP+L4jhBDIs+IAfFy7VeA7hXf4ZaChox8pwHX hQ6x32GqP5n+EzlLacdDQ==
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 15:30:21 -0000

On 2013-11-06 16:50, Julian Reschke wrote:
> 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

...and applied in 
<http://trac.tools.ietf.org/wg/httpbis/trac/changeset/2480>.

Best regards, Julian