Date: Sun, 11 Nov 2018 02:01:47 -0800
From: Tatsuhiro Tsujikawa <notifications@github.com>
Reply-To: quicwg/base-drafts
 <reply+0166e4abf089ec4c7bb40e7da5e99471fc05d9922461855792cf0000000117ffc00b92a169ce169d83c2@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/issues/1987/437657121@github.com>
In-Reply-To: <quicwg/base-drafts/issues/1987@github.com>
References: <quicwg/base-drafts/issues/1987@github.com>
Subject: Re: [quicwg/base-drafts] Clarify 1200 bytes Initial packet rule
 (#1987)
Mime-Version: 1.0
Content-Type: multipart/alternative;
 boundary="--==_mimepart_5be7fe0bc0bd1_72593fadf8cd45c0419756";
 charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/D-3StHP0Y4dxVCNIBo32dQyWlq0>


----==_mimepart_5be7fe0bc0bd1_72593fadf8cd45c0419756
Content-Type: text/plain;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

Regarding when to stop padding Initial:

> Clients MUST pad UDP datagrams that contain only Initial packets to 1200 bytes. Once a client has received an acknowledgment for a Handshake packet it MAY send smaller datagrams. Sending padded datagrams ensures that the server is not overly constrained by the amplification restriction.

I think that client can stop padding Initial when it has CRYPTO frame to send in Handshake packet.  This is because at this time server has nothing to send and client will retransmit Handshake packet thus no dead lock.


-- 
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/1987#issuecomment-437657121
----==_mimepart_5be7fe0bc0bd1_72593fadf8cd45c0419756
Content-Type: text/html;
 charset=UTF-8
Content-Transfer-Encoding: quoted-printable

<p>Regarding when to stop padding Initial:</p>
<blockquote>
<p>Clients MUST pad UDP datagrams that contain only Initial packets to 12=
00 bytes. Once a client has received an acknowledgment for a Handshake pa=
cket it MAY send smaller datagrams. Sending padded datagrams ensures that=
 the server is not overly constrained by the amplification restriction.</=
p>
</blockquote>
<p>I think that client can stop padding Initial when it has CRYPTO frame =
to send in Handshake packet.  This is because at this time server has not=
hing to send and client will retransmit Handshake packet thus no dead loc=
k.</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/1987#issuecomment-437657121">view it on GitHub</a>,=
 or <a href=3D"https://github.com/notifications/unsubscribe-auth/AWbkq99A=
k_NyYCa9-HtFGU4QRfye35qJks5ut_WLgaJpZM4YX-za">mute the thread</a>.<img sr=
c=3D"https://github.com/notifications/beacon/AWbkqwnIRdNnnrORRg8lJB8zR8XT=
v99Fks5ut_WLgaJpZM4YX-za.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":"@tats=
uhiro-t in #1987: Regarding when to stop padding Initial:\r\n\r\n\u003e C=
lients MUST pad UDP datagrams that contain only Initial packets to 1200 b=
ytes. Once a client has received an acknowledgment for a Handshake packet=
 it MAY send smaller datagrams. Sending padded datagrams ensures that the=
 server is not overly constrained by the amplification restriction.\r\n\r=
\nI think that client can stop padding Initial when it has CRYPTO frame t=
o send in Handshake packet.  This is because at this time server has noth=
ing to send and client will retransmit Handshake packet thus no dead lock=
.\r\n"}],"action":{"name":"View Issue","url":"https://github.com/quicwg/b=
ase-drafts/issues/1987#issuecomment-437657121"}}}</script>
<script type=3D"application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/quicwg/base-drafts/issues/1987#issuecomment=
-437657121",
"url": "https://github.com/quicwg/base-drafts/issues/1987#issuecomment-43=
7657121",
"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] Clarify 1200 bytes Initial packet rule=
 (#1987)",
"sections": [
{
"text": "",
"activityTitle": "**Tatsuhiro Tsujikawa**",
"activityImage": "https://assets-cdn.github.com/images/email/message_card=
s/avatar.png",
"activitySubtitle": "@tatsuhiro-t",
"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\": 1987,\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\": 1987\n}"
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/quicwg/base-drafts/issues/1987#issuecomment-43=
7657121"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 4089397=
38\n}"
}
],
"themeColor": "26292E"
}
]</script>=

----==_mimepart_5be7fe0bc0bd1_72593fadf8cd45c0419756--