Date: Wed, 04 Jul 2018 17:39:35 -0700
From: Martin Thomson <notifications@github.com>
Reply-To: quicwg/base-drafts
 <reply+0166e4aba627c301b423dca9452f4ffb801fe4af322af77592cf0000000117552ac792a169ce1427404b@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/issues/1518/402576715@github.com>
In-Reply-To: <quicwg/base-drafts/issues/1518@github.com>
References: <quicwg/base-drafts/issues/1518@github.com>
Subject: Re: [quicwg/base-drafts] Make EOED transmission optional in QUIC,
 please (#1518)
Mime-Version: 1.0
Content-Type: multipart/alternative;
 boundary="--==_mimepart_5b3d68c76ab56_2a422ab0c8c40f601391280";
 charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/RTqfA9N0wPj4JJByLzPMUGmDd3g>


----==_mimepart_5b3d68c76ab56_2a422ab0c8c40f601391280
Content-Type: text/plain;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

It says "handshake data", which is correct, if perhaps a little too easy to mistake for "Handshake packets".

Acknowledging promptly is primarily an optimization.  It should still work with the leisurely timers.  My naive assumption was that when you process a CRYPTO frame you would cut the ack delay timer.  That gets you a pretty good approximation of the right behaviour, even if that also means that NewSessionTicket forces more aggressive acknowledgment than necessary (for which you could condition on the packet protection level if you cared).

-- 
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/1518#issuecomment-402576715
----==_mimepart_5b3d68c76ab56_2a422ab0c8c40f601391280
Content-Type: text/html;
 charset=UTF-8
Content-Transfer-Encoding: quoted-printable

<p>It says "handshake data", which is correct, if perhaps a little too ea=
sy to mistake for "Handshake packets".</p>
<p>Acknowledging promptly is primarily an optimization.  It should still =
work with the leisurely timers.  My naive assumption was that when you pr=
ocess a CRYPTO frame you would cut the ack delay timer.  That gets you a =
pretty good approximation of the right behaviour, even if that also means=
 that NewSessionTicket forces more aggressive acknowledgment than necessa=
ry (for which you could condition on the packet protection level if you c=
ared).</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/1518#issuecomment-402576715">view it on GitHub</a>,=
 or <a href=3D"https://github.com/notifications/unsubscribe-auth/AWbkq7es=
yyrlTLBZPD-CGSzKNoWfxsEPks5uDWBHgaJpZM4VB5Yx">mute the thread</a>.<img sr=
c=3D"https://github.com/notifications/beacon/AWbkq8L9dxsXgG5sGHtecCt1H_Di=
EuIdks5uDWBHgaJpZM4VB5Yx.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":"@mart=
inthomson in #1518: It says \"handshake data\", which is correct, if perh=
aps a little too easy to mistake for \"Handshake packets\".\r\n\r\nAcknow=
ledging promptly is primarily an optimization.  It should still work with=
 the leisurely timers.  My naive assumption was that when you process a C=
RYPTO frame you would cut the ack delay timer.  That gets you a pretty go=
od approximation of the right behaviour, even if that also means that New=
SessionTicket forces more aggressive acknowledgment than necessary (for w=
hich you could condition on the packet protection level if you cared)."}]=
,"action":{"name":"View Issue","url":"https://github.com/quicwg/base-draf=
ts/issues/1518#issuecomment-402576715"}}}</script>
<script type=3D"application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/quicwg/base-drafts/issues/1518#issuecomment=
-402576715",
"url": "https://github.com/quicwg/base-drafts/issues/1518#issuecomment-40=
2576715",
"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] Make EOED transmission optional in QUI=
C, please (#1518)",
"sections": [
{
"text": "",
"activityTitle": "**Martin Thomson**",
"activityImage": "https://assets-cdn.github.com/images/email/message_card=
s/avatar.png",
"activitySubtitle": "@martinthomson",
"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\": 1518,\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\": 1518\n}"
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/quicwg/base-drafts/issues/1518#issuecomment-40=
2576715"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 3528187=
37\n}"
}
],
"themeColor": "26292E"
}
]</script>=

----==_mimepart_5b3d68c76ab56_2a422ab0c8c40f601391280--