Date: Sat, 15 Jul 2017 17:28:33 +0000 (UTC)
From: ianswett <notifications@github.com>
Reply-To: quicwg/base-drafts
 <reply+0166e4ab08aa696b791dcae918a18092e8d83a280f31faf492cf00000001158212c192a169ce0e7ea8ec@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/685/review/50193681@github.com>
In-Reply-To: <quicwg/base-drafts/pull/685@github.com>
References: <quicwg/base-drafts/pull/685@github.com>
Subject: Re: [quicwg/base-drafts] Change Client Initial size to 1200 flat
 (#685)
Mime-Version: 1.0
Content-Type: multipart/alternative;
 boundary="--==_mimepart_596a50c138056_1ad23fc91cb6dc385511c";
 charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/0UdaK40N-WNgeUcygZGI7r3aujQ>

----==_mimepart_596a50c138056_1ad23fc91cb6dc385511c
Content-Type: text/plain;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

ianswett commented on this pull request.



> @@ -541,7 +541,7 @@ older than 1.3 is negotiated.
 
 QUIC requires that the initial handshake packet from a client fit within the
 payload of a single packet.  The size limits on QUIC packets mean that a record
-containing a ClientHello needs to fit within 1197 octets.
+containing a ClientHello needs to fit within 1165 octets.

To be clear, this 1165 number may change if we increase or decrease the amount of framing overhead QUIC adds.  Are we ok about that?

-- 
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/pull/685#pullrequestreview-50193681
----==_mimepart_596a50c138056_1ad23fc91cb6dc385511c
Content-Type: text/html;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

<p><b>@ianswett</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/quicwg/base-drafts/pull/685#discussion_r127589005">draft-ietf-quic-tls.md</a>:</p>
<pre style='color:#555'>&gt; @@ -541,7 +541,7 @@ older than 1.3 is negotiated.
 
 QUIC requires that the initial handshake packet from a client fit within the
 payload of a single packet.  The size limits on QUIC packets mean that a record
-containing a ClientHello needs to fit within 1197 octets.
+containing a ClientHello needs to fit within 1165 octets.
</pre>
<p>To be clear, this 1165 number may change if we increase or decrease the amount of framing overhead QUIC adds.  Are we ok about that?</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/quicwg/base-drafts/pull/685#pullrequestreview-50193681">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AWbkqyr0ALFDxToPT5vwt21jmm3nkRD2ks5sOPbBgaJpZM4OZCzd">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AWbkq1T_OwBfZM3Wz8YREB2R7vV9BdMUks5sOPbBgaJpZM4OZCzd.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/quicwg/base-drafts/pull/685#pullrequestreview-50193681"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"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":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/quicwg/base-drafts"}},"updates":{"snippets":[{"icon":"PERSON","message":"@ianswett commented on #685"}],"action":{"name":"View Pull Request","url":"https://github.com/quicwg/base-drafts/pull/685#pullrequestreview-50193681"}}}</script>
----==_mimepart_596a50c138056_1ad23fc91cb6dc385511c--