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.

Sign up for GitHub

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

Jump to bottom

Switch to using cloudflare-ech.com as the target for the ech test #1658

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Switch to using cloudflare-ech.com as the target for the ech test #1658

wants to merge 4 commits into from

Conversation

hellais
Copy link
Member

@hellais hellais commented Nov 7, 2024 •
edited
Loading

Checklist

Description

Changes to the ECHCheck experiment.

  • Replace default URL with cloudflare-ech.com
  • Add support for performing an additional ECH handshake with a different ClientHelloOuter SNI
  • Randomize the order of the handshakes

@hellais hellais requested a review from DecFox as a code owner November 7, 2024 13:11
@hellais
Update echcheck test to 0.2.0 placing tls_handshakes into list
3fa848b 
* Update TLSHandshake model to include ECHConfig field
* Update tests
@hellais
Copy link
Member Author

hellais commented Nov 7, 2024

For updates to the spec see: ooni/spec#297

@hellais
Add support for OuterSni field
186df09 
* Add extra handshake that uses a different outer SNI field
@hellais hellais mentioned this pull request Nov 8, 2024
Open
@hellais hellais mentioned this pull request Nov 11, 2024
Open
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DecFox DecFox Awaiting requested review from DecFox DecFox is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@hellais