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

fix torsf #1529

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

fix torsf #1529

wants to merge 3 commits into from

Conversation

Lanius-collaris
Copy link
Contributor

@Lanius-collaris Lanius-collaris commented Mar 22, 2024

Checklist

  • I have read the contribution guidelines
  • reference issue for this pull request: snowflake: 421 Misdirected Request probe#2692
  • if you changed anything related to how experiments work and you need to reflect these changes in the ooni/spec repository, please link to the related ooni/spec pull request:
  • if you changed code inside an experiment, make sure you bump its version number

Description

Please, insert here a more detailed description.
https://gitlab.torproject.org/tpo/applications/tor-browser-build/-/commit/0554d981793e6beb280cd8bd12e01081ca0e4c59

@bassosimone
Copy link
Contributor

This feels like a good solution and idea. I'll add some changes to make it not dirty. Thank you so much for proposing this solution, as I was struggling to put the context together and figuring out a way of upgrading dependencies. You definitely saved the day! 🙏🙏🙏🙏

bassosimone added a commit that referenced this pull request Mar 25, 2024
This diff updates snowflake's domain fronting such that `torsf` is
working as intended again. I adapted this diff from
#1529, written by @Lanius-collaris
(thank you so much! 🥰 🙌). The original diff contained extra
functionality that it would be very nice to have. I have chosen to drop
this functionality, because I'd like to quickly merge a Snowflake fix. I
will take care of the additional changes in a subsequent diff.

Part of ooni/probe#2692.

---------

Co-authored-by: Lanius-collaris <[email protected]>
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.

2 participants