IETF Logo Mail Archive

Re: [quicwg/base-drafts] Consider making long header payload length 16bits (#1577)

Nick Banks <notifications@github.com> Wed, 18 July 2018 21:20 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 70D9C131060 for <quic-issues@ietfa.amsl.com>; Wed, 18 Jul 2018 14:20:41 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -8.009
X-Spam-Level:
X-Spam-Status: No, score=-8.009 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, URIBL_BLOCKED=0.001] 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 e4WABub-SbJR for <quic-issues@ietfa.amsl.com>; Wed, 18 Jul 2018 14:20:39 -0700 (PDT)
Received: from out-2.smtp.github.com (out-2.smtp.github.com [192.30.252.193]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 0D6BE131052 for <quic-issues@ietf.org>; Wed, 18 Jul 2018 14:20:39 -0700 (PDT)
Date: Wed, 18 Jul 2018 14:20:38 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1531948838; bh=+poTInuaLVU/tijOEsIvDVtOANpKFZWHQGo9iEEBpBI=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=il/Ovw9DokT4LPxX8gupaOQU4yqtaLRfIukFYGT7WI4GeH8AihWz5zWxc4ysEWezu LkNW9Nn2iY9Xl8RHo90QtBOXb6Oew1RJhOWLXxlXcVeMMpYIIjxEuJXDxLuLZcMW8Z yTLzXd9KkZRfh8TRGu3x7r24c/op3u6a9XTH3uyw=
From: Nick Banks <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4ab65cdf45a9471e7f537fa8ae23ff7dc14ccd86e5192cf000000011767712692a169ce146220ed@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/issues/1577/406078137@github.com>
In-Reply-To: <quicwg/base-drafts/issues/1577@github.com>
References: <quicwg/base-drafts/issues/1577@github.com>
Subject: Re: [quicwg/base-drafts] Consider making long header payload length 16bits (#1577)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5b4faf265aca6_35123fd7828ccf50120264"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: nibanks
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/bVtLpwsRQCzGQ_fwABuh0QmyLJ4>
X-BeenThere: quic-issues@ietf.org
X-Mailman-Version: 2.1.27
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: Wed, 18 Jul 2018 21:20:41 -0000

I disagree that changing the length to a 2-byte field increases complexity. Maybe things aren't quite as consistent, but the code to read the field isn't going to drastically increase in complexity because of it. It would likely get a tad bit simpler too. Additionally, I don't think we need to worry about allowing for the _possibility_ that someone really wants to save a single byte in a long header (only during the handshake).

-- 
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/1577#issuecomment-406078137

v2.23.0 | Report a Bug | By Email