IETF Logo Mail Archive

Re: [quicwg/base-drafts] Reorder HTTP frame fields to put Type first (#1351)

Lucas Pardue <notifications@github.com> Fri, 11 May 2018 14:26 UTC

Return-Path: <noreply@github.com>
X-Original-To: quic-issues@ietfa.amsl.com
Delivered-To: quic-issues@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 8FDD912D945 for <quic-issues@ietfa.amsl.com>; Fri, 11 May 2018 07:26:40 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -8.01
X-Spam-Level:
X-Spam-Status: No, score=-8.01 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-0.001, T_DKIMWL_WL_HIGH=-0.01] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=github.com
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id ksGXB14nvW3l for <quic-issues@ietfa.amsl.com>; Fri, 11 May 2018 07:26:39 -0700 (PDT)
Received: from out-6.smtp.github.com (out-6.smtp.github.com [192.30.252.197]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 0CB5F12D942 for <quic-issues@ietf.org>; Fri, 11 May 2018 07:26:38 -0700 (PDT)
Date: Fri, 11 May 2018 07:26:37 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1526048797; bh=x3izf52XPlWWD1lQZeCiyc0qaEoNK1ETXJE6OLERk6Y=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=0l0sekhqwbtD/JThr8ZltadWG6YsLFOXqCVW91kQvpPu223JXY/IJPtWz1+OSJQxB xuM2EZURiQceOnIMwlfSXjOosgvsbt9Bjw/tf5Jm0BywqxSzo3L37JdMqnDhksamZ9 WEXlk53y+DdbwDXmYYEKZtResXoQFzc/yGcc/Bok=
From: Lucas Pardue <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4abd288ce783c90d39c5fea79638366aec0861e68f092cf00000001170d6a1d92a169ce1335e6ee@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/issues/1351/388379729@github.com>
In-Reply-To: <quicwg/base-drafts/issues/1351@github.com>
References: <quicwg/base-drafts/issues/1351@github.com>
Subject: Re: [quicwg/base-drafts] Reorder HTTP frame fields to put Type first (#1351)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5af5a81d5e7d2_94b2af1335e6f60345172"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: LPardue
X-GitHub-Recipient: quic-issues
X-GitHub-Reason: subscribed
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address: quic-issues@ietf.org
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/T7jdJhXj7Sae8B5Q4_TQJxTfeZg>
X-BeenThere: quic-issues@ietf.org
X-Mailman-Version: 2.1.22
List-Id: Notification list for GitHub issues related to the QUIC WG <quic-issues.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/quic-issues>, <mailto:quic-issues-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/quic-issues/>
List-Post: <mailto:quic-issues@ietf.org>
List-Help: <mailto:quic-issues-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/quic-issues>, <mailto:quic-issues-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 11 May 2018 14:26:40 -0000

The frame layout is close to HTTP/2. However it IS already different
because it strips the Stream ID.

There is unlikely to be much reuse between h2 parsers and hq frame parsers.

On Fri, 11 May 2018, 15:09 Dirkjan Ochtman, <notifications@github.com>
wrote:

> Why is that better? I don't think I necessarily agree. Having to pass the
> length to the type-specific parser makes the interface different from the
> one in other places, as explained, and more tightly bound.
>
> Additionally, I prefer having variable-sized fields after types of
> statically known size.
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/quicwg/base-drafts/issues/1351#issuecomment-388374835>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AGRFtVTt8JYOXFx2e4vdez6j9Kji8OnUks5txZungaJpZM4T7eOr>
> .
>


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/quicwg/base-drafts/issues/1351#issuecomment-388379729

v2.31.3 | Report a Bug | By Email | System Status