1
72%
master: 72%

Ran 20 Sep 2023 11:47PM UTC

Files 504

Run time 8s

Badge

Embed ▾

Committed 20 Sep 2023 11:41PM UTC coverage: 81.494% (-0.02%) from 81.514%

Job # 6255251967.1

Build Type

push

github

Committed by

web-flow

Commit Message

refactor: adapt to netem pinning certificates to hosts (#1287)

As explained in https://github.com/ooni/netem/pull/40, netem used to
generate on the fly certificates for any host.

I then discovered that this behavior is not desirable because the
internet doesn't work like that. Specifically, TLS dialing to, say,
www.example.com with www.google.com as the SNI must return a TLS error
(`ssl_invalid_hostname` in OONI).

This wrong behavior of netem has become a bottleneck for writing new
tests for the beacons functionality, so it must change.

I have already changed netem and this diff updates the probe-cli tree to
use a netem version that behaves more correctly.

While there, I noticed that sniblocking tests were wrong because of the
previous netem behavior and asserted that the control should return a
nil failure, while it should have been `ssl_invalid_hostname`.

Part of https://github.com/ooni/probe/issues/2531

Run Details

23459 of 28786 relevant lines covered (81.49%)

34.06 hits per line

ooni / probe-cli / 6255251967 / 1
72%
master: 72%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 0 - 6255251967.1

ooni / probe-cli / 6255251967 / 1 72% master: 72%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 0 - 6255251967.1

ooni / probe-cli / 6255251967 / 1
72%
master: 72%

README BADGES
x