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

webconnectivity: false positive with lgbt.foundation #1734

Closed
bassosimone opened this issue Aug 17, 2021 · 2 comments · Fixed by ooni/probe-cli#1476
Closed

webconnectivity: false positive with lgbt.foundation #1734

bassosimone opened this issue Aug 17, 2021 · 2 comments · Fixed by ooni/probe-cli#1476
Assignees
Labels

Comments

@bassosimone
Copy link
Contributor

This happens because of Cloudflare browser checks and has been reported in our Slack channels.

@bassosimone bassosimone added bug Something isn't working triage labels Aug 17, 2021
@bassosimone bassosimone self-assigned this Aug 17, 2021
@arky
Copy link

arky commented Aug 17, 2021

bassosimone added a commit to ooni/spec that referenced this issue Jan 30, 2024
## Checklist

- [x] I have read the [contribution
guidelines](https://github.com/ooni/spec/blob/master/CONTRIBUTING.md)
- [x] reference issue for this pull request:
ooni/probe#2661
- [x] related ooni/probe-cli pull request:
ooni/probe-cli#1476
- [x] If I changed a spec, I also bumped its version number and/or date

## Description

SSB means server-side blocking.

See ooni/probe#2661

See ooni/probe-cli#1476

See ooni/probe#1734
@bassosimone
Copy link
Contributor Author

So, I have added test case for this specific situation and documented a bit more in detail what the problem is and what the next steps could be in #2661. I am going to close this issue with the understanding that the underlying problem described in here has been generalized, explained, and tested. The tests for now verify that we say "http-diff" with http:// like URLs, which ensures we continue to produce consistent behavior. Solving the problem of server-side blocking is a broader discussion that encompasses not just the case shown here but more broadly a wide variety of cases. Because we have several data quality issues related to this, I am going to try and consolidate these issues in #2661 to make project management and decision processes easier.

bassosimone added a commit to ooni/probe-cli that referenced this issue Jan 30, 2024
## Checklist

- [x] I have read the [contribution
guidelines](https://github.com/ooni/probe-cli/blob/master/CONTRIBUTING.md)
- [x] reference issue for this pull request:
ooni/probe#1734
- [x] if you changed anything related to how experiments work and you
need to reflect these changes in the ooni/spec repository, please link
to the related ooni/spec pull request:
ooni/spec#284
- [x] if you changed code inside an experiment, make sure you bump its
version number: not needed

## Description

Closes ooni/probe#1734.

See ooni/probe#2661.

While there, rename an incorrectly renamed Yandex test.

While there, ignore the `/oonireport` binary.
Murphy-OrangeMud pushed a commit to Murphy-OrangeMud/probe-cli that referenced this issue Feb 13, 2024
## Checklist

- [x] I have read the [contribution
guidelines](https://github.com/ooni/probe-cli/blob/master/CONTRIBUTING.md)
- [x] reference issue for this pull request:
ooni/probe#1734
- [x] if you changed anything related to how experiments work and you
need to reflect these changes in the ooni/spec repository, please link
to the related ooni/spec pull request:
ooni/spec#284
- [x] if you changed code inside an experiment, make sure you bump its
version number: not needed

## Description

Closes ooni/probe#1734.

See ooni/probe#2661.

While there, rename an incorrectly renamed Yandex test.

While there, ignore the `/oonireport` binary.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants