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

Invariants in Transport Parameters #1672

Closed
mirjak opened this issue Aug 16, 2018 · 2 comments
Closed

Invariants in Transport Parameters #1672

mirjak opened this issue Aug 16, 2018 · 2 comments
Assignees
Labels
-invariants -transport editorial An issue that does not affect the design of the protocol; does not require consensus.

Comments

@mirjak
Copy link
Contributor

mirjak commented Aug 16, 2018

The transport draft says:
"When an endpoint accepts multiple QUIC versions, it can potentially
interpret transport parameters as they are defined by any of the QUIC
versions it supports. The version field in the QUIC packet header is
authenticated using transport parameters. The position and the
format of the version fields in transport parameters MUST either be
identical across different QUIC versions, or be unambiguously
different to ensure no confusion about their interpretation. One way
that a new format could be introduced is to define a TLS extension
with a different codepoint."
It seems that this actually belongs in the invariants draft.

@MikeBishop
Copy link
Contributor

It's not actually an invariant, because it's not true of all QUIC versions, nor is it (for servers, at least) visible outside the encryption envelope. It's really advice to the authors of future QUIC versions about how to avoid bugs when supporting both versions simultaneously. (There's similar text on how to design a multi-version Stateless Reset.)

@MikeBishop MikeBishop changed the title Invarinats in TP Invariants in TP Aug 16, 2018
@MikeBishop MikeBishop changed the title Invariants in TP Invariants in Transport Parameters Aug 16, 2018
@martinthomson martinthomson added the editorial An issue that does not affect the design of the protocol; does not require consensus. label Aug 17, 2018
@martinthomson
Copy link
Member

I don't think that we need to do anything about this, and I would be opposed to moving this to invariants for the reasons @MikeBishop outlines. I would consider a PR that improves the text.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-invariants -transport editorial An issue that does not affect the design of the protocol; does not require consensus.
Projects
None yet
Development

No branches or pull requests

3 participants