Date: Mon, 11 Nov 2019 15:55:27 -0800
From: Eric Kinnear <notifications@github.com>
Reply-To: quicwg/base-drafts
 <reply+AFTOJK4FVNTNOA65RMYAQ5N324TW7EVBNHHB5Y6ONQ@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/issues/3193/552671227@github.com>
In-Reply-To: <quicwg/base-drafts/issues/3193@github.com>
References: <quicwg/base-drafts/issues/3193@github.com>
Subject: Re: [quicwg/base-drafts] active_connection_id_limit interacts poorly
 with Retire Prior To (#3193)
Mime-Version: 1.0
Content-Type: multipart/alternative;
 boundary="--==_mimepart_5dc9f4ef4617_417d3fb3c22cd96c29390c9";
 charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/RHIolbPTrV0a9I7gVe8utqA_lns>


----==_mimepart_5dc9f4ef4617_417d3fb3c22cd96c29390c9
Content-Type: text/plain;
 charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Right, I think the issues about packets that are already enqueued, etc. a=
re more for the person requesting retirement of the CID to know that some=
 packets might still show up (although potentially not after the ACK for =
the NCID frame that had RPT if they're all in the same queue? Not sure I'=
d want to rely on that ordering even before looking at the network). The =
endpoint that is retiring the CID can simply stop sending _new_ packets w=
ith that CID and be fine.=0D
=0D
And leaving it up to the implementation but making a note that it's not a=
 problem to retire/replace right up to the limit sounds good to me. =F0=9F=
=91=8D=0D
=0D
-- =0D
You are receiving this because you are subscribed to this thread.=0D
Reply to this email directly or view it on GitHub:=0D
https://github.com/quicwg/base-drafts/issues/3193#issuecomment-552671227=

----==_mimepart_5dc9f4ef4617_417d3fb3c22cd96c29390c9
Content-Type: text/html;
 charset=UTF-8
Content-Transfer-Encoding: quoted-printable

<p>Right, I think the issues about packets that are already enqueued, etc=
. are more for the person requesting retirement of the CID to know that s=
ome packets might still show up (although potentially not after the ACK f=
or the NCID frame that had RPT if they're all in the same queue? Not sure=
 I'd want to rely on that ordering even before looking at the network). T=
he endpoint that is retiring the CID can simply stop sending <em>new</em>=
 packets with that CID and be fine.</p>=0D
<p>And leaving it up to the implementation but making a note that it's no=
t a problem to retire/replace right up to the limit sounds good to me. <g=
-emoji class=3D"g-emoji" alias=3D"+1" fallback-src=3D"https://github.gith=
ubassets.com/images/icons/emoji/unicode/1f44d.png">=F0=9F=91=8D</g-emoji>=
</p>=0D
=0D
<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/3193?email_source=3Dnotifications&amp;email_token=3D=
AFTOJKZAZGEV3XKT6GYOQDTQTHWG7A5CNFSM4JJPZITKYY3PNVWWK3TUL52HS4DFVREXG43VM=
VBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDYRP6Y#issuecomment-552671227">view it on=
 GitHub</a>, or <a href=3D"https://github.com/notifications/unsubscribe-a=
uth/AFTOJK35N5DMK4I2SA2OB2TQTHWG7ANCNFSM4JJPZITA">unsubscribe</a>.<img sr=
c=3D"https://github.com/notifications/beacon/AFTOJK374TDC35EF67BWQRLQTHWG=
7A5CNFSM4JJPZITKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWS=
ZGOEDYRP6Y.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/issues/3193?email_source=
=3Dnotifications\u0026email_token=3DAFTOJKZAZGEV3XKT6GYOQDTQTHWG7A5CNFSM4=
JJPZITKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDYRP6=
Y#issuecomment-552671227",=0D
"url": "https://github.com/quicwg/base-drafts/issues/3193?email_source=3D=
notifications\u0026email_token=3DAFTOJKZAZGEV3XKT6GYOQDTQTHWG7A5CNFSM4JJP=
ZITKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDYRP6Y#i=
ssuecomment-552671227",=0D
"name": "View Issue"=0D
},=0D
"description": "View this Issue on GitHub",=0D
"publisher": {=0D
"@type": "Organization",=0D
"name": "GitHub",=0D
"url": "https://github.com"=0D
}=0D
}=0D
]</script>=

----==_mimepart_5dc9f4ef4617_417d3fb3c22cd96c29390c9--