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

Spindump should recognise failed version negotiations attempts #170

Open
jariarkko opened this issue Feb 4, 2020 · 1 comment
Open

Spindump should recognise failed version negotiations attempts #170

jariarkko opened this issue Feb 4, 2020 · 1 comment
Labels
enhancement New feature or request

Comments

@jariarkko
Copy link
Collaborator

Test case trace_quic_v25_ngx for instance is one such case.

(But can we theoretically recognise these? Unclear)

@jariarkko jariarkko added the enhancement New feature or request label Feb 4, 2020
@MaximProshin
Copy link
Contributor

According to the quic transport draft version 25 a server responds by the Version Negotiation packet if a client selected an unsupported version. The client then must abandon the current connection attempt and may initiate a new connection using one of the versions supported by the server. From that I understood that the spindump tool just needs to recognize the Version Negotiation packet, report the version negotiation fail if needed and close the connection.
One corner case here is that a server may decide to not send the Version Negotiation packet (or it might be lost) but it seems to be more general problem for the spindump tool.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants