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

Invariant: the first packet on a connection uses the long header #1040

Closed
martinthomson opened this issue Jan 9, 2018 · 5 comments
Closed

Comments

@martinthomson
Copy link
Member

We need this for version negotiation to work.

@MikeBishop
Copy link
Contributor

Agreed -- this is generally assumed, but not actually written down.

@ianswett
Copy link
Contributor

ianswett commented Jan 9, 2018

In order for this not to be true, that implies you have done out of band version negotiation and key negotiation? At that point, I would call it migration rather than a new connection.

One use case is to migrate a connection from a TLS 1.3 TCP connection to a QUIC connection. A similar use might be WebRTC using QUIC?

It seems like we could do these no matter what we decide here.

@martinthomson
Copy link
Member Author

Yes. There's always this little annoying insect that reminds me that people start QUIC from not-QUIC with keys and context already established. Those people are welcome to do so, but they aren't really using the same protocol and so wouldn't be bound by the same constraints. In this case, they would have to understand that they don't get to use version negotiation.

martinthomson added a commit that referenced this issue Jan 10, 2018
@martinthomson
Copy link
Member Author

Closed in e6ac5c1.

@MikeBishop
Copy link
Contributor

And if they need it, all they have to do is define a version that has the long-header 1-RTT packet types put back.

martinthomson added a commit that referenced this issue Jan 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants