Date: Wed, 18 Jul 2018 14:20:38 -0700
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
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/bVtLpwsRQCzGQ_fwABuh0QmyLJ4>


----==_mimepart_5b4faf265aca6_35123fd7828ccf50120264
Content-Type: text/plain;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

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
----==_mimepart_5b4faf265aca6_35123fd7828ccf50120264
Content-Type: text/html;
 charset=UTF-8
Content-Transfer-Encoding: quoted-printable

<p>I disagree that changing the length to a 2-byte field increases comple=
xity. Maybe things aren't quite as consistent, but the code to read the f=
ield 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 ne=
ed to worry about allowing for the <em>possibility</em> that someone real=
ly wants to save a single byte in a long header (only during the handshak=
e).</p>

<p style=3D"font-size:small;-webkit-text-size-adjust:none;color:#666;">&m=
dash;<br />You are receiving this because you are subscribed to this thre=
ad.<br />Reply to this email directly, <a href=3D"https://github.com/quic=
wg/base-drafts/issues/1577#issuecomment-406078137">view it on GitHub</a>,=
 or <a href=3D"https://github.com/notifications/unsubscribe-auth/AWbkq4ke=
KJ1B5Cpul9Ir9zcszg_X6eZAks5uH6amgaJpZM4VTFuj">mute the thread</a>.<img sr=
c=3D"https://github.com/notifications/beacon/AWbkq20iWa3cNzPv2f7y7xlzvbch=
6nsJks5uH6amgaJpZM4VTFuj.gif" height=3D"1" width=3D"1" alt=3D"" /></p>
<script type=3D"application/json" data-scope=3D"inboxmarkup">{"api_versio=
n":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name"=
:"GitHub"},"entity":{"external_key":"github/quicwg/base-drafts","title":"=
quicwg/base-drafts","subtitle":"GitHub repository","main_image_url":"http=
s://assets-cdn.github.com/images/email/message_cards/header.png","avatar_=
image_url":"https://assets-cdn.github.com/images/email/message_cards/avat=
ar.png","action":{"name":"Open in GitHub","url":"https://github.com/quicw=
g/base-drafts"}},"updates":{"snippets":[{"icon":"PERSON","message":"@niba=
nks in #1577: I disagree that changing the length to a 2-byte field incre=
ases 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 hand=
shake)."}],"action":{"name":"View Issue","url":"https://github.com/quicwg=
/base-drafts/issues/1577#issuecomment-406078137"}}}</script>
<script type=3D"application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/quicwg/base-drafts/issues/1577#issuecomment=
-406078137",
"url": "https://github.com/quicwg/base-drafts/issues/1577#issuecomment-40=
6078137",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
},
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "Re: [quicwg/base-drafts] Consider making long header payload le=
ngth 16bits (#1577)",
"sections": [
{
"text": "",
"activityTitle": "**Nick Banks**",
"activityImage": "https://assets-cdn.github.com/images/email/message_card=
s/avatar.png",
"activitySubtitle": "@nibanks",
"facts": [

]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueComment\",\n\"repositoryFullName\": \=
"quicwg/base-drafts\",\n\"issueId\": 1577,\n\"IssueComment\": \"{{IssueCo=
mment.value}}\"\n}"
}
]
},
{
"name": "Close issue",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueClose\",\n\"repositoryFullName\": \"q=
uicwg/base-drafts\",\n\"issueId\": 1577\n}"
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/quicwg/base-drafts/issues/1577#issuecomment-40=
6078137"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 3573257=
31\n}"
}
],
"themeColor": "26292E"
}
]</script>=

----==_mimepart_5b4faf265aca6_35123fd7828ccf50120264--