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

Draft: Obfs4 handshakes #2

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Draft: Obfs4 handshakes #2

wants to merge 8 commits into from

Conversation

cyBerta
Copy link
Owner

@cyBerta cyBerta commented Mar 20, 2023

beginning from 3673185
built on top of #1

Checklist

  • I have read the contribution guidelines
  • reference issue for this pull request:
  • 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:
  • if you changed code inside an experiment, make sure you bump its version number

Description

Please, insert here a more detailed description.

cyBerta added 8 commits March 6, 2023 15:17
…r the ca was invalid. Instead of early returning the test, we can still gather the necessary configs to run the gateway tests. This change implies that there might occur multiple failures while fetching data from the API, which is why TestKeys.APIFailures becomes a string array.
… longer skipped, API config data is fetched even if CA cert is invalid or missing
…4 bridges, avoid hard-coding locations in the test
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