Date: Tue, 29 Oct 2019 15:20:46 +0000 (UTC)
From: Mike Bishop <notifications@github.com>
Reply-To: quicwg/base-drafts
 <reply+AFTOJK7FLW2QOB4JQTOGA3F3YWMV5EVBNHHB4QFASQ@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/3096/review/308596243@github.com>
In-Reply-To: <quicwg/base-drafts/pull/3096@github.com>
References: <quicwg/base-drafts/pull/3096@github.com>
Subject: Re: [quicwg/base-drafts] MUST retire Connection IDs becoming stale
 (#3096)
Mime-Version: 1.0
Content-Type: multipart/alternative;
 boundary="--==_mimepart_5db858ce66e22_f5a3fe5ebecd9642684be";
 charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/nlAv4Nmple_A85J15hmeGFcUp-U>

----==_mimepart_5db858ce66e22_f5a3fe5ebecd9642684be
Content-Type: text/plain;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

MikeBishop approved this pull request.

@mikkelfj, the delayed packets are handled by this text:

>  Tokens are invalidated when their associated connection ID is retired via a RETIRE_CONNECTION_ID frame (Section 19.16).

> An endpoint MUST NOT check for any Stateless Reset Tokens associated with connection IDs it has not used or for connection IDs that have been retired.

If you've followed the requirement to retire as indicated, then a delayed packet triggering stateless reset can't hurt you.

> @@ -1032,16 +1032,19 @@ longer plans to use that address.
 
 An endpoint can request that its peer retire connection IDs by sending a

This isn't really a request any more, is it?
```suggestion
An endpoint can cause its peer to retire connection IDs by sending a
```

-- 
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/3096#pullrequestreview-308596243
----==_mimepart_5db858ce66e22_f5a3fe5ebecd9642684be
Content-Type: text/html;
 charset=UTF-8
Content-Transfer-Encoding: quoted-printable

<p><b>@MikeBishop</b> approved this pull request.</p>=0D
=0D
<p><a class=3D"user-mention" data-hovercard-type=3D"user" data-hovercard-ur=
l=3D"/hovercards?user_id=3D193335" data-octo-click=3D"hovercard-link-click"=
 data-octo-dimensions=3D"link_type:self" href=3D"https://github.com/mikkelf=
j">@mikkelfj</a>, the delayed packets are handled by this text:</p>=0D
<blockquote>=0D
<p>Tokens are invalidated when their associated connection ID is retired vi=
a a RETIRE_CONNECTION_ID frame (Section 19.16).</p>=0D
</blockquote>=0D
<blockquote>=0D
<p>An endpoint MUST NOT check for any Stateless Reset Tokens associated wit=
h connection IDs it has not used or for connection IDs that have been retir=
ed.</p>=0D
</blockquote>=0D
<p>If you've followed the requirement to retire as indicated, then a delaye=
d packet triggering stateless reset can't hurt you.</p><hr>=0D
=0D
<p>In <a href=3D"https://github.com/quicwg/base-drafts/pull/3096#discussion=
_r340141146">draft-ietf-quic-transport.md</a>:</p>=0D
<pre style=3D'color:#555'>&gt; @@ -1032,16 +1032,19 @@ longer plans to use =
that address.=0D
 =0D
 An endpoint can request that its peer retire connection IDs by sending a=
=0D
</pre>=0D
<p>This isn't really a request any more, is it?</p>=0D
=E2=AC=87=EF=B8=8F Suggested change=0D
<pre style=3D"color: #555">- An endpoint can request that its peer retire c=
onnection IDs by sending a=0D
+An endpoint can cause its peer to retire connection IDs by sending a=0D
</pre>=0D
=0D
=0D
<p style=3D"font-size:small;-webkit-text-size-adjust:none;color:#666;">&mda=
sh;<br />You are receiving this because you are subscribed to this thread.<=
br />Reply to this email directly, <a href=3D"https://github.com/quicwg/bas=
e-drafts/pull/3096?email_source=3Dnotifications&amp;email_token=3DAFTOJK7JL=
AFH5S6FXGSUGODQRBIE5A5CNFSM4JBEWMH2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2F=
EZLWNFSXPKTDN5WW2ZLOORPWSZGOCJSM4EY#pullrequestreview-308596243">view it on=
 GitHub</a>, or <a href=3D"https://github.com/notifications/unsubscribe-aut=
h/AFTOJKZR2EKSXFJ64APGWK3QRBIE5ANCNFSM4JBEWMHQ">unsubscribe</a>.<img src=3D=
"https://github.com/notifications/beacon/AFTOJK56NLSGQ2XDRV456ADQRBIE5A5CNF=
SM4JBEWMH2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWS=
ZGOCJSM4EY.gif" height=3D"1" width=3D"1" alt=3D"" /></p>=0D
<script type=3D"application/ld+json">[=0D
{=0D
"@context": "http://schema.org",=0D
"@type": "EmailMessage",=0D
"potentialAction": {=0D
"@type": "ViewAction",=0D
"target": "https://github.com/quicwg/base-drafts/pull/3096?email_source=3Dn=
otifications\u0026email_token=3DAFTOJK7JLAFH5S6FXGSUGODQRBIE5A5CNFSM4JBEWMH=
2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCJSM4E=
Y#pullrequestreview-308596243",=0D
"url": "https://github.com/quicwg/base-drafts/pull/3096?email_source=3Dnoti=
fications\u0026email_token=3DAFTOJK7JLAFH5S6FXGSUGODQRBIE5A5CNFSM4JBEWMH2YY=
3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCJSM4EY#p=
ullrequestreview-308596243",=0D
"name": "View Pull Request"=0D
},=0D
"description": "View this Pull Request on GitHub",=0D
"publisher": {=0D
"@type": "Organization",=0D
"name": "GitHub",=0D
"url": "https://github.com"=0D
}=0D
}=0D
]</script>=

----==_mimepart_5db858ce66e22_f5a3fe5ebecd9642684be--