Skip to content

Revert "Feature/im tests"#298

Open
drexer006 wants to merge 387 commits into
ooni:add-to-captive-portalfrom
drexer006:revert-79-feature/im-tests
Open

Revert "Feature/im tests"#298
drexer006 wants to merge 387 commits into
ooni:add-to-captive-portalfrom
drexer006:revert-79-feature/im-tests

Conversation

@drexer006

Copy link
Copy Markdown

Reverts #79

darkk and others added 30 commits November 7, 2018 00:23
Add techniques helping to distinguish network outage from filtering.
Adds techniques from PATHspider
Add text about possibility second response is firewalled
No functional change, just prettification, hence I think it's fine for me to commit this directly on master.
It's not JSON. It looks like it, but it triggers parse errors on the GitHub side.

Again, it's fine to commit on master this microdiff.
1. explicitly mention how the summary variables are computed, because that was left implicit

2. modify the spec to make it consistent with all existing implementations with respect to when we consider telegram to be blocked at the HTTP level (the previous wording was such that I could not decide what it meant). Both MK and ooni/probe-legacy start with setting Telegram blocked at the HTTP level and then unblock it if "successful". (I am profoundly skeptical that we can easily define success here without speaking the protocol, and without validating certificates, because in reality we may be speaking with any endpoint; also, in my limited tests, now Telegram does not respond with `501` anymore -- all of this tells us possible ways in which we can perhaps improve our Telegram implementation)
Readme.md: add a minimal explanation
Fix telegram spec to increase clarity
* Write v2.0.0 bouncer spc

* Update bk-004-bouncer.md

* Update bk-004-bouncer.md

* Update bk-004-bouncer.md

* Update bk-004-bouncer.md

* Address feedback by @hellais
bassosimone and others added 28 commits April 3, 2023 15:47
)

Example of a measurement showing this behaviour: https://explorer.ooni.org/m/20210101003640.413492_AR_telegram_ad086fbac780bdae

---------

Co-authored-by: Simone Basso <bassosimone@gmail.com>
This pull request syncs riseupvpn spec with ooniprobe 3.19.

Closes ooni/probe#2563.

---------

Co-authored-by: Simone Basso <bassosimone@gmail.com>
We had a conversation today where we concluded to clearly mark some fields we're currently experimenting with as experimental. Obviously, we don't plan on changing them for the sake of change, but we also want to be careful about communicating in which areas we're still experimenting.

Part of ooni/ooni.org#1384
This diff documents the `ooni_run_link_id` annotation.
## 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#2627
- [x] related ooni/probe-cli pull request: ooni/probe-cli#1389
- [x] If I changed a spec, I also bumped its version number and/or date

## Description

This updates the signal test spec to the changes to the tested endpoints
## 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#2636
- [x] related ooni/probe-cli pull request:
ooni/probe-cli#1421
- [x] If I changed a spec, I also bumped its version number and/or date

<!-- Location of the issue tracker: https://github.com/ooni/probe -->

## Description

We trim down signal endpoints to increase robustness against endpoints
churn.
## 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#1539
- [x] related ooni/probe-cli pull request: N/A
- [x] If I changed a spec, I also bumped its version number and/or date:
not needed

## Description

Closes ooni/probe#1539 by clarifying the
semantics of server_name.
## 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
…oni#285)

## 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#2628
- [x] related ooni/probe-cli pull request:
ooni/probe-cli#1480
- [x] If I changed a spec, I also bumped its version number and/or date:
not needed
In markdown, there should be just one top-level entry in a file.

To address this, create a title with the nettest name and then make sure
all the other titles fall below one level.

Part of ooni/probe#2666

(If I have time, I'll try to do some maintenance of all the nettest
specs and implement similar changes, but for now I will only limit my
boyscout-rules instincts to the spec I am editing.)
I think the spec was lacking too much detail and did not explain to a
technical user the big picture of what happens when they run Web
Connectivity.

I know this choice is going to put more burden on the maintainer of the
spec but I am the maintainer and I am happy about carrying this burden.

See ooni/probe#2666.
The examples regarding the data format take lots of space and make
reading the specification difficult possibly causing readers to miss
important sections available near the end.

As a fix, move all the examples at the end in its own section.

See ooni/probe#2666
This diff updates the measurement algorithm description to describe Web
Connectivity v0.5.

Part of ooni/probe#2666
the previous phrasing could induce to think that `t` is only measured in
the case of failures.

## Checklist

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

- [x] I have read the [contribution
guidelines](https://github.com/ooni/spec/blob/master/CONTRIBUTING.md)
- [x] ooni/probe#2742
- [ ] related ooni/probe-cli pull request: <!-- add URL here -->
- [ ] If I changed a spec, I also bumped its version number and/or date

## Description

This diff updates the telegram nettest spec to include the latest IP
endpoints we measure as part of the test

Co-authored-by: decfox <decfox@decfoxs-MBP.lan>
## 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#2688
- [x] related ooni/probe-cli pull request:
ooni/probe-cli#1585
- [x] If I changed a spec, I also bumped its version number and/or date



## Description

Add initial spec for the `openvpn` experiment.
This proposal supersedes a previous version
(ooni#255).
ICMP Pings and URLGrabber over the tunnel are not included, this
experiment just does OpenVPN handshakes at the moment.

---------

Co-authored-by: Simone Basso <bassosimone@gmail.com>
…i#295)

## 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#2758
- [ ] related ooni/probe-cli pull request: <!-- add URL here -->
- [x] If I changed a spec, I also bumped its version number and/or date


## Description

Modify spec to comply with the semantics of df-009-tunnel
## Checklist

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

<!-- Location of the issue tracker: https://github.com/ooni/probe -->

## Description

Please, insert here a more detailed description.
@drexer006
drexer006 changed the base branch from master to DPI-Fragmentation-Spec July 22, 2026 08:47
@drexer006
drexer006 changed the base branch from DPI-Fragmentation-Spec to add-to-captive-portal July 22, 2026 08:48
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.