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

Only allow whitelisted protocols in tor mode #1378

Closed
jumde opened this issue Oct 1, 2018 · 3 comments · Fixed by brave/brave-core#583
Closed

Only allow whitelisted protocols in tor mode #1378

jumde opened this issue Oct 1, 2018 · 3 comments · Fixed by brave/brave-core#583

Comments

@jumde
Copy link
Contributor

jumde commented Oct 1, 2018

brave/browser-laptop#14664
brave/browser-laptop#14665

@jumde jumde added the feature/tor/leakproofing Eliminating unexpected ways that someone using Tor might be unmasked. label Oct 1, 2018
@diracdeltas
Copy link
Member

I just remembered this led to an easy IP proxy bypass on MacOS, so it should be marked as release blocking.

@diracdeltas diracdeltas added this to the Releasable builds 0.55.x milestone Oct 1, 2018
@diracdeltas diracdeltas changed the title Allow whitelisted protocols in tor mode Only allow whitelisted protocols in tor mode Oct 1, 2018
@riastradh-brave riastradh-brave self-assigned this Oct 3, 2018
@yrliou yrliou self-assigned this Oct 4, 2018
@yrliou yrliou added the QA/Yes label Oct 9, 2018
@yrliou
Copy link
Member

yrliou commented Oct 9, 2018

Test plan specified in brave/brave-core#583

@btlechowski
Copy link

btlechowski commented Oct 10, 2018

Verification passed on

Brave 0.55.12 Chromium: 70.0.3538.45 (Official Build) (64-bit)
Revision cbdc32e4334458954e9def214d7e5fa1ca1960eb-refs/branch-heads/3538@{#830}
OS Windows 7

Used test plan from brave/brave-core#583

Verification Passed on

Brave 0.55.12 Chromium: 70.0.3538.45 (Official Build) beta (64-bit)
Revision cbdc32e4334458954e9def214d7e5fa1ca1960eb-refs/branch-heads/3538@{#830}
OS Linux

image

Verified passed on

Brave 0.55.12 Chromium: 70.0.3538.45 (Official Build) beta(64-bit)
Revision cbdc32e4334458954e9def214d7e5fa1ca1960eb-refs/branch-heads/3538@{#830}
OS Mac OS X

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment