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

snowflake: 421 Misdirected Request #2692

Closed
1 task done
bassosimone opened this issue Mar 21, 2024 · 1 comment
Closed
1 task done

snowflake: 421 Misdirected Request #2692

bassosimone opened this issue Mar 21, 2024 · 1 comment
Assignees

Comments

@bassosimone
Copy link
Contributor

bassosimone commented Mar 21, 2024

Snowflake is now failing with 421 Misdirected Request because the domain_fronting strategy does not work anymore. We need to upgrade Snowflake to fix this issue. We should try to do this as part of the 3.21 release cycle.

As part of handling this, I am disabling Snowflake's integration tests. TODO(@bassosimone):

  • enable Snowflake integration tests when this is fixed
@bassosimone bassosimone self-assigned this Mar 21, 2024
@bassosimone bassosimone mentioned this issue Mar 21, 2024
36 tasks
bassosimone added a commit to ooni/backend that referenced this issue Mar 21, 2024
This diff uses richer input to disable the torsf experiment. This works under
the assumption that probes periodically call check-in and refresh feature flags,
which should be true for the probes maintained by the OONI team.

Mitigation for ooni/probe#2692
bassosimone added a commit to ooni/probe-cli that referenced this issue Mar 22, 2024
bassosimone added a commit to ooni/probe-cli that referenced this issue Mar 22, 2024
hellais pushed a commit to ooni/backend that referenced this issue Mar 25, 2024
This diff uses richer input to disable the torsf experiment. This works under
the assumption that probes periodically call check-in and refresh feature flags,
which should be true for the probes maintained by the OONI team.

Mitigation for ooni/probe#2692
bassosimone added a commit to ooni/probe-cli that referenced this issue 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 <55432068+Lanius-collaris@users.noreply.github.com>
@bassosimone
Copy link
Contributor Author

Thanks for @Lanius-collaris, snowflake is now working as intended again. There will still be the question of how to correctly upgrade it and use a version that is compatible with Psiphon, but that's a different issue.

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

1 participant