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

fix(wcth): emit empty Addrs when input URL contains addr #545

Merged
merged 1 commit into from
Oct 15, 2021

Conversation

bassosimone
Copy link
Contributor

@bassosimone bassosimone commented Oct 15, 2021

Checklist

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

Description

Matches the behavior that the legacy TH implements in this
situation and reduces slightly the differences.

See ooni/probe#1707 (comment)

Matches the behavior that the legacy TH implements in this
situation and reduces slightly the differences.

See ooni/probe#1707 (comment)
@bassosimone bassosimone requested a review from hellais as a code owner October 15, 2021 09:43
@bassosimone bassosimone merged commit 2d1666b into master Oct 15, 2021
@bassosimone bassosimone deleted the issue/1707 branch October 15, 2021 10:00
bassosimone added a commit that referenced this pull request Oct 20, 2021
Matches the behavior that the legacy TH implements in this
situation and reduces slightly the differences.

See ooni/probe#1707 (comment)

Backport from master; see ooni/probe#1839
ainghazal pushed a commit to ainghazal/probe-cli that referenced this pull request Mar 8, 2022
Matches the behavior that the legacy TH implements in this
situation and reduces slightly the differences.

See ooni/probe#1707 (comment)
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