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

Add text describing deployment impacts of no SNI access #566

Merged
merged 9 commits into from
Oct 13, 2023

Conversation

chris-wood
Copy link
Collaborator

@chris-wood chris-wood commented Oct 10, 2023

This lifts text from the ECH deployment considerations draft for inclusion in the ECH draft. It is not uncommon for protocol documents to address deployment considerations head on, and this PR attempts to do so. As always, feedback on the exact phrasing is welcome.

cc @dennisjackson, @davidben, @martinthomson, @cjpatton, @sftcd

draft-ietf-tls-esni.md Outdated Show resolved Hide resolved
draft-ietf-tls-esni.md Outdated Show resolved Hide resolved
draft-ietf-tls-esni.md Outdated Show resolved Hide resolved
chris-wood and others added 2 commits October 10, 2023 16:18
Co-authored-by: Christopher Patton <[email protected]>
Co-authored-by: Christopher Patton <[email protected]>
@dennisjackson
Copy link
Contributor

LGTM. Do you want to add something about untrusted environments? e.g.

"In an environment where the network operator controls the endpoint devices, but is concerned about malware or similar exfiltrating data, the SNI field is unsuitable for use as a control even in the absence of ECH. This is because malware is able to spoof the value in an SNI field already, and can even fool security appliances which try to 'double-check' the websites hosted by the target server. ECH does not materially change this situation."

@chris-wood
Copy link
Collaborator Author

@dennisjackson I worked in your suggested paragraph. Please take another look!

@dennisjackson
Copy link
Contributor

@dennisjackson I worked in your suggested paragraph. Please take another look!

Two small suggestions, otherwise LGTM.

chris-wood and others added 2 commits October 13, 2023 06:32
Co-authored-by: Dennis Jackson <[email protected]>
Co-authored-by: Dennis Jackson <[email protected]>
draft-ietf-tls-esni.md Outdated Show resolved Hide resolved
draft-ietf-tls-esni.md Outdated Show resolved Hide resolved
@chris-wood chris-wood merged commit c08b4e6 into master Oct 13, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants