Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Does PMTU fail really want unauthenticated PUBLIC_RESET? #327

Closed
mcmanus opened this issue Feb 20, 2017 · 2 comments
Closed

Does PMTU fail really want unauthenticated PUBLIC_RESET? #327

mcmanus opened this issue Feb 20, 2017 · 2 comments
Labels
-transport invalid A duplicate, overcome-by-events, ill-formed, or off-topic issue, or a question better asked on-list.

Comments

@mcmanus
Copy link
Contributor

mcmanus commented Feb 20, 2017

section 8 of transport:`

If a QUIC endpoint determines that the PMTU between any pair of local and remote IP addresses has fallen below 1280 octets, it MUST immediately cease sending QUIC packets between those IP addresses. This may result in abrupt termination of the connection if all pairs are affected. In this case, an endpoint SHOULD send a Public Reset packet to indicate the failure.

but PR is defined as being a last resort and it is of course not authenticated.. while it is short, a CONNECTION_CLOSE only packet would also be below minimum MTUs for IPvmumble, and is authenticated and could carry some diagnostic information.

It seems like this error can be authenticated and therefore should be.. and more generally we shouldn't promote error reporting to RST as its interaction with already sent data is very unclear if TCP RST is any kind of corollary.
`

@RyanTheOptimist
Copy link
Contributor

RyanTheOptimist commented Feb 20, 2017 via email

@martinthomson
Copy link
Member

Duplicate of #289.

(FWIW, my reading of the list is that this is unequivocally good.)

@mnot mnot added the invalid A duplicate, overcome-by-events, ill-formed, or off-topic issue, or a question better asked on-list. label Mar 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-transport invalid A duplicate, overcome-by-events, ill-formed, or off-topic issue, or a question better asked on-list.
Projects
None yet
Development

No branches or pull requests

4 participants