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

Replace nonfunctional fastly front with cdn77 fronts #1197

Merged
merged 1 commit into from
Nov 12, 2024

Conversation

cohosh
Copy link
Contributor

@cohosh cohosh commented Nov 4, 2024

Related to #1191 and https://gitlab.torproject.org/tpo/anti-censorship/team/-/issues/151

Although, this change isn't necessary because, despite being used to call IPtProxy's startSnowflake, this target/front configuration is never used and overridden by the Snowflake bridge lines.

This is only really useful if there is a bridge line without a specified front.

@syphyr
Copy link
Contributor

syphyr commented Nov 7, 2024

snowflake-target needs to have a "/" at the end according to the source code. It should look like this:

snowflake-target https://1098762253.rsc.cdn77.org/

@cohosh
Copy link
Contributor Author

cohosh commented Nov 7, 2024

It doesn't matter. See some of our other example settings: https://bridges.torproject.org/moat/circumvention/map

This value is used for the url parameter in the bridge line. You can test it yourself by trying the following bridge line:

Bridge snowflake 192.0.2.3:80 2B280B23E1107BB62ABFC40DDCC8824814F80A72 fingerprint=2B280B23E1107BB62ABFC40DDCC8824814F80A72 url=https://1098762253.rsc.cdn77.org fronts=www.cdn77.com,www.phpmyadmin.net ice=stun:stun.l.google.com:19302,stun:stun.antisip.com:3478,stun:stun.bluesip.net:3478,stun:stun.dus.net:3478,stun:stun.epygi.com:3478,stun:stun.sonetel.com:3478,stun:stun.uls.co.za:3478,stun:stun.voipgate.com:3478,stun:stun.voys.nl:3478 utls-imitate=hellorandomizedalpn

@n8fr8 n8fr8 merged commit 438d3d4 into guardianproject:master Nov 12, 2024
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