-
Notifications
You must be signed in to change notification settings - Fork 46
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: use probe-cli v3.19.0 #622
Conversation
@aanorbel it's 3.19.0-alpha |
…ved from the `CLI`
Fixes ooni/probe#2561 ## Proposed Changes - Update support for socks proxy to use dropdown for selecting a scheme. - Update code to save and retrieve the various proxies. ![Screenshot_20231013_171055](https://github.com/ooni/probe-android/assets/17911892/2bad783f-6570-478d-9754-f07a17184664)
if (pm == null || pm.isTestRiseupVPN()) | ||
list.add(new RiseupVPN());*/ | ||
if (pm == null || pm.isTestRiseupVPN()) | ||
list.add(new RiseupVPN()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should run RiseupVPN inside the experimental suite. If this change requires too much churn, I would recommend enabling it in the next release, but it must not run as part of circumvention.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Started a PR to effect this #625 .
We would need to test properly and merge before the next release
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🐳
Part of ooni/probe#2524
Proposed Changes
echcheck
mobile: add echcheck experiment probe#2567.