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

refactor(netxlite): adapt single-use-quic-dialer from websteps #472

Merged
merged 1 commit into from
Sep 6, 2021

Conversation

bassosimone
Copy link
Contributor

@bassosimone bassosimone commented Sep 6, 2021

Checklist

Location of the issue tracker: https://github.com/ooni/probe

Description

This is the last bit of functionality we need before rewriting a
chunk of websteps to use netxlite.

To rewrite most of it, we still need to move over:

  1. dnstransport code

  2. errorsx code

With both done, netxlite is a good library for websteps as well
as for most other operations we perform outside of the experiments.

Part of ooni/probe#1591

This is the last bit of functionality we need before rewriting a
chunk of websteps to use netxlite.

To rewrite most of it, we still need to move over:

1. dnstransport code

2. errorsx code

With both done, netxlite is a good library for websteps as well
as for most other operations we perform outside of the experiments.

Part of ooni/probe#1591
@bassosimone bassosimone requested a review from hellais as a code owner September 6, 2021 20:04
@bassosimone bassosimone merged commit fe3c904 into master Sep 6, 2021
@bassosimone bassosimone deleted the issue/1591 branch September 6, 2021 20:14
ainghazal pushed a commit to ainghazal/probe-cli that referenced this pull request Mar 8, 2022
)

This is the last bit of functionality we need before rewriting a
chunk of websteps to use netxlite.

To rewrite most of it, we still need to move over:

1. dnstransport code

2. errorsx code

With both done, netxlite is a good library for websteps as well
as for most other operations we perform outside of the experiments.

Part of ooni/probe#1591
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.

1 participant