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

desktop, cli, mobile: tell users about bad proxies #416

Open
bassosimone opened this issue Oct 16, 2023 · 0 comments
Open

desktop, cli, mobile: tell users about bad proxies #416

bassosimone opened this issue Oct 16, 2023 · 0 comments
Assignees
Labels
bug Something isn't working correctly funder/otffoss2025 priority/low Nice to have

Comments

@bassosimone
Copy link

This issue is about telling users that they have configured a bad proxy. For example, suppose tor is listening at socks5://127.0.0.1:9050/ and I mistype it as socks5://127.0.0.2:9050/, it would be nice for me, as a user, to know that the OONI Probe engine cannot boostrap because of such a typo.

There are two obvious approaches here. The first approach is wrapping proxy errors such that they are recognizable by the specific OONI Probe App. The second approach is testing the proxy right after it is configured.

The problem with the first approach is that "all available probe services failed", the error that is returned when the proxy is not working, is a composed error containing an error for each attempt. Therefore, to conclude that we have a proxy error, we would need somehow to aggregate to conclude something like "proxy error: all available probe services failed".

The problem with testing a proxy right after it has been configured is that a proxy may work at time T1 but may stop working at time T2. For example, I may be using Orbot and Orbot may be stopped some time in the future. So, when I configure the proxy at time T1 all looks good, but we subsequently see a failure at time T2 when I use it.

An additional approach could be for the OONI Engine to test the proxy before using it, when it is configured to use a proxy. If we factor in the case of tor, torsf, and psiphon, this would mean testing the proxy after the tor, torsf, or psiphon bootstrap.

If we go down with this approach, we would have a preliminary step, which consists of checking whether we can use the proxy and which fails with an informative error in case the proxy does not work. Obviously, this means that, if the proxy fails at a later time, we will have several failures, but at least we can say it was working when we started measuring.

@bassosimone bassosimone added the bug Something isn't working correctly label Oct 16, 2023
@bassosimone bassosimone self-assigned this Oct 16, 2023
@hellais hellais added priority/low Nice to have and removed priority/medium Normal priority issue labels Jan 23, 2025
@hellais hellais assigned sdsantos and aanorbel and unassigned bassosimone Jan 23, 2025
@sdsantos sdsantos transferred this issue from ooni/probe Jan 27, 2025
@hellais hellais removed this from Roadmap Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working correctly funder/otffoss2025 priority/low Nice to have
Projects
None yet
Development

No branches or pull requests

5 participants