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

Extraneous configurations MUST have invalid DNS names? #630

Open
ekr opened this issue Oct 12, 2024 · 4 comments
Open

Extraneous configurations MUST have invalid DNS names? #630

ekr opened this issue Oct 12, 2024 · 4 comments

Comments

@ekr
Copy link
Collaborator

ekr commented Oct 12, 2024

Suggested in AD review.

@ekr
Copy link
Collaborator Author

ekr commented Oct 12, 2024

Also, should we provide guidance on how to select these names. Paul writes:

Should it use known-invalid DNS names, eg "invalid:com", or some randomized
long valid but unlikely DNS name? Guidaance would be useful.

@ekr
Copy link
Collaborator Author

ekr commented Oct 27, 2024

@bemasc

@bemasc
Copy link
Contributor

bemasc commented Oct 28, 2024

This text is from @davidben here: #569 (comment)

The goal of this recommendation is to catch clients who are not respecting the "mandatory" bit and force them to fail hard. To do this, the server provides an ECHConfig that is syntactically well-formed but unusable (due to a reserved mandatory extension).

Using a syntactically invalid domain name would defeat the purpose, because clients would discard the ECHConfig without inspecting the extensions. Instead, the server should choose a public_name that is syntactically valid but for which it is not authoritative.

@davidben notes that a name under .invalid would work. This would be a fine choice so long as clients don't carry special logic to detect and reject these names.

@seanturner
Copy link
Contributor

At IETF 121, decided to use .invalid.

ekr added a commit to ekr/draft-ietf-tls-esni that referenced this issue Nov 24, 2024
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

3 participants