Re: Comments on draft-ietf-httpbis-http2-04

Jeff Pinner <jpinner@twitter.com> Tue, 09 July 2013 19:51 UTC

Return-Path: <ietf-http-wg-request@listhub.w3.org>
X-Original-To: ietfarch-httpbisa-archive-bis2Juki@ietfa.amsl.com
Delivered-To: ietfarch-httpbisa-archive-bis2Juki@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 6671C11E814D for <ietfarch-httpbisa-archive-bis2Juki@ietfa.amsl.com>; Tue, 9 Jul 2013 12:51:49 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -7.462
X-Spam-Level:
X-Spam-Status: No, score=-7.462 tagged_above=-999 required=5 tests=[AWL=-2.486, BAYES_00=-2.599, FM_FORGED_GMAIL=0.622, GB_SUMOF=5, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-8]
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 7yJVoiLRTuRR for <ietfarch-httpbisa-archive-bis2Juki@ietfa.amsl.com>; Tue, 9 Jul 2013 12:51:42 -0700 (PDT)
Received: from frink.w3.org (frink.w3.org [128.30.52.56]) by ietfa.amsl.com (Postfix) with ESMTP id BAA9911E8143 for <httpbisa-archive-bis2Juki@lists.ietf.org>; Tue, 9 Jul 2013 12:51:39 -0700 (PDT)
Received: from lists by frink.w3.org with local (Exim 4.72) (envelope-from <ietf-http-wg-request@listhub.w3.org>) id 1UwdwJ-0005ox-FU for ietf-http-wg-dist@listhub.w3.org; Tue, 09 Jul 2013 19:51:07 +0000
Resent-Date: Tue, 09 Jul 2013 19:51:07 +0000
Resent-Message-Id: <E1UwdwJ-0005ox-FU@frink.w3.org>
Received: from lisa.w3.org ([128.30.52.41]) by frink.w3.org with esmtp (Exim 4.72) (envelope-from <jpinner@twitter.com>) id 1UwdwC-0005ny-6D for ietf-http-wg@listhub.w3.org; Tue, 09 Jul 2013 19:51:00 +0000
Received: from mail-oa0-f51.google.com ([209.85.219.51]) by lisa.w3.org with esmtps (TLS1.0:RSA_ARCFOUR_SHA1:16) (Exim 4.72) (envelope-from <jpinner@twitter.com>) id 1UwdwB-0002jc-1s for ietf-http-wg@w3.org; Tue, 09 Jul 2013 19:51:00 +0000
Received: by mail-oa0-f51.google.com with SMTP id i4so8531957oah.10 for <ietf-http-wg@w3.org>; Tue, 09 Jul 2013 12:50:33 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=twitter.com; s=google; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=HPTRSsz+eneCQrLeSrJ2psIVAYI6duR8vKq2LjL2lF4=; b=maL8j8p+1MKC8Kgnn7YQw68sZJtV1J5WMqa50FKTdA8guAn4ihUGChF/SZp83XcaAI nwYBLUZ0yzZL2JtS6dSUkGsbGTfXnnGJ2bvMsbRUW2dK1nOriMkXOamCvgwO5RdfY131 R8QD6ztjk1OguuplD3bacCztdEMpowWIiyC9M=
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:x-gm-message-state; bh=HPTRSsz+eneCQrLeSrJ2psIVAYI6duR8vKq2LjL2lF4=; b=OIN73T3YEPkjxDz6qSRMX7F+xDSEpGbUu9T0+h/mm9BbBNDTD/lKGM0SWz3Fqw0m1P vFYzUMZ8uBUk9bcYbwG/KlIhMJbjrvD1xOlY74/f2L6N8SN1ESmQrwB4DT2Te2y06Ypn 7UczGRmU35FVPacG2AkoaroPdN+5RQjmT30ts1IPJnfbTFF+W0QFjsXyrSawS6QS3ohZ ODm6hdielkcnfJWp3Gr6f81g+hEMnGhC4dzTCpDWEX1wzc7EP3YLPxy5+4qkZNFXooEy ZWkNIzfFzPVo++mv7172L247nkjHKzkSih37nSqyTmoMyXifB7eIjTYNpZPnAWvLMko/ MDaQ==
MIME-Version: 1.0
X-Received: by 10.182.144.231 with SMTP id sp7mr25767352obb.14.1373399433109; Tue, 09 Jul 2013 12:50:33 -0700 (PDT)
Received: by 10.182.7.37 with HTTP; Tue, 9 Jul 2013 12:50:33 -0700 (PDT)
In-Reply-To: <CABkgnnWexuQb9vZPudJTJ+Gk0LAtcunWG1fThrk3Y_Eo9mDv=A@mail.gmail.com>
References: <3072E3B4-63B4-4DFB-AFD8-08EE6407C6FB@apple.com> <CABkgnnWexuQb9vZPudJTJ+Gk0LAtcunWG1fThrk3Y_Eo9mDv=A@mail.gmail.com>
Date: Tue, 09 Jul 2013 12:50:33 -0700
Message-ID: <CA+pLO_gzNTpTabeuXE7SE+J8Bnx7ky3bnxdKLxB5A-DiAS01Uw@mail.gmail.com>
From: Jeff Pinner <jpinner@twitter.com>
To: Martin Thomson <martin.thomson@gmail.com>
Cc: Michael Sweet <msweet@apple.com>, "ietf-http-wg@w3.org list" <ietf-http-wg@w3.org>
Content-Type: multipart/alternative; boundary="089e0153686c587a7704e119799b"
X-Gm-Message-State: ALoCoQmW4BKmCV9y7l6NPM+PSn2TQ/5E40D5CDcFaOOIhfsP+C0Z+BrWwuRkQUhPyNRN86lJI4RT
Received-SPF: pass client-ip=209.85.219.51; envelope-from=jpinner@twitter.com; helo=mail-oa0-f51.google.com
X-W3C-Hub-Spam-Status: No, score=-3.9
X-W3C-Hub-Spam-Report: AWL=-3.100, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-0.7, SPF_PASS=-0.001
X-W3C-Scan-Sig: lisa.w3.org 1UwdwB-0002jc-1s 08fb21cd5c21d082f255188dd7892c36
X-Original-To: ietf-http-wg@w3.org
Subject: Re: Comments on draft-ietf-httpbis-http2-04
Archived-At: <http://www.w3.org/mid/CA+pLO_gzNTpTabeuXE7SE+J8Bnx7ky3bnxdKLxB5A-DiAS01Uw@mail.gmail.com>
Resent-From: ietf-http-wg@w3.org
X-Mailing-List: <ietf-http-wg@w3.org> archive/latest/18660
X-Loop: ietf-http-wg@w3.org
Resent-Sender: ietf-http-wg-request@w3.org
Precedence: list
List-Id: <ietf-http-wg.w3.org>
List-Help: <http://www.w3.org/Mail/>
List-Post: <mailto:ietf-http-wg@w3.org>
List-Unsubscribe: <mailto:ietf-http-wg-request@w3.org?subject=unsubscribe>

I would prefer we say something to the effect of:

If a server receives a request with a "Content-Length" header and
the sum of the DATA frame payload lengths does not equal the value
of the "content-length" header field, the server MUST return a 400 (Bad
Request) error.

and remove any other MUSTs and SHOULDs. By the way, I would like to note
that this is different than how the HTTP/1.1 spec handles
Tranfser-Encoding. From RFC2616 Sec 4.4:

If a message is received with both a Transfer-Encoding header field and a
Content-Length header field, the latter MUST be ignored.

- Jeff


On Tue, Jul 9, 2013 at 12:40 PM, Martin Thomson <martin.thomson@gmail.com>wrote:

> On 9 July 2013 10:12, Michael Sweet <msweet@apple.com> wrote:
> > I am uncomfortable with this wording primarily because a lot of POST
> usage
> > consists of streamed content - my particular interest is obviously
> printing,
> > but any streamed content will necessarily not be able to provide a
> > content-length header field. So instead I would suggest the following two
> > paragraphs instead:
>
> I think that your edits capture the spirit of what was intended by the
> existing text, with far less ambiguity.  Unless I get objections,
> those can be integrated.
>
> > My other comment is that I don't see any discussion of the Expect header,
> > nor do I see a issue on Github...
>
> There was a brief discussion at the last interim.  The feature is in
> serious jeopardy.  See #18:
> https://github.com/http2/http2-spec/issues/18
>
>