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

chore: update psiphon #1522

Merged
merged 3 commits into from
Mar 25, 2024
Merged

chore: update psiphon #1522

merged 3 commits into from
Mar 25, 2024

Conversation

bassosimone
Copy link
Contributor

@bassosimone bassosimone commented Mar 21, 2024

It seems there's an issue by which some replace directives in the go.mod used by Psiphon aren't picked up by us. The thing is that replace only works for the main module. We thus asked for clarification but we can merge this PR anyway, since it helps us to find dependency intersection between snowflake and psiphon (I am thinking mainly in terms of refraction-networking/utls).

See ooni/probe#2690

It seems there's an issue by which some `replace` directives in
the `go.mod` used by Psiphon aren't picked up by us. The thing is
that `replace` only works for the main module. We thus asked for
clarification before merging this PR.

See ooni/probe#2690
@bassosimone bassosimone mentioned this pull request Mar 21, 2024
36 tasks
@bassosimone
Copy link
Contributor Author

I am going to merge, because I need this to understand whether we can intersect psiphon and snowflake as dependencies.

I took note in the release check list of the implications of the replace directive.

@bassosimone bassosimone marked this pull request as ready for review March 25, 2024 13:05
@bassosimone bassosimone requested a review from hellais as a code owner March 25, 2024 13:05
@bassosimone bassosimone merged commit 3ab3f89 into master Mar 25, 2024
28 checks passed
@bassosimone bassosimone deleted the issue/2690a branch March 25, 2024 13:18
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.

1 participant