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

Add switch to disable WebRTC #551

Closed
jumde opened this issue Jul 13, 2018 · 11 comments · Fixed by brave/brave-core#538
Closed

Add switch to disable WebRTC #551

jumde opened this issue Jul 13, 2018 · 11 comments · Fixed by brave/brave-core#538
Assignees
Labels

Comments

@jumde
Copy link
Contributor

jumde commented Jul 13, 2018

No description provided.

@bbondy bbondy added this to the Backlog milestone Jul 16, 2018
@bbondy
Copy link
Member

bbondy commented Jul 16, 2018

Is this different from fingerprinting protection which I think already does this?

@jumde
Copy link
Contributor Author

jumde commented Jul 16, 2018

Block All fingerprinting does disable WebRTC but, after a reddit discussion the webrtc options was moved to a site setting in case the user does not Block All Fingerprinting since it does break some known sites. See: brave/browser-laptop#13668

@jumde
Copy link
Contributor Author

jumde commented Jul 17, 2018

https://whoer.net/ - Shows WebRTC leaks even with Block All Fingerprinting

@yrliou
Copy link
Member

yrliou commented Jul 17, 2018

Block All fingerprinting does not disable WebRTC entirely, it only make methods that could be causing IP leaks act similar as no-op.
I suppose the original idea of adding this switch in browser-laptop is users could have a way to disable WebRTC without turning on Block All Fingerprinting which might break sites.

I just quickly checked the website, the result is as follows.
screen shot 2018-07-16 at 5 37 04 pm
It does prevent IP leakage and has the same result of Block All fingerprinting in browser-laptop.
@jumde Could you point me where the test showing us having the WebRTC leak so I could check on my end?

@jumde
Copy link
Contributor Author

jumde commented Jul 17, 2018

@yrliou - WebRTC is blocked on whoer.net, I think the page did not reload correctly, and so it was still showing the IP address. Sorry about that.

Re: -

I suppose the original idea of adding this switch in browser-laptop is users could have a way to disable WebRTC without turning on Block All Fingerprinting which might break sites.

We should probably do the same in brave-core

@jumde
Copy link
Contributor Author

jumde commented Aug 16, 2018

This is available on browser-laptop

screen shot 2018-08-16 at 1 51 53 pm

@eljuno
Copy link

eljuno commented Sep 7, 2018

+1 from community https://community.brave.com/t/brave-dev-is-excellent-with-chrome-addons-but-what-addons-you-use/31625?u=eljuno

Brave Dev doesnot have control over webRTC leak as Brave regular.

@Brave-Matt
Copy link

@bbondy bbondy modified the milestones: 1.x Backlog, Releasable builds 0.55.x Sep 28, 2018
@yrliou yrliou self-assigned this Sep 29, 2018
@rebron rebron added the design A design change, especially one which needs input from the design team label Sep 30, 2018
@rebron
Copy link
Collaborator

rebron commented Sep 30, 2018

@bradleyrichter We have another setting that needs a landing place. Right below safe browsing and before crash reports.
screen shot 2018-09-30 at 10 16 38 am

In between these two settings looks to make the most sense.
screen shot 2018-09-30 at 10 18 27 am

@yrliou
Copy link
Member

yrliou commented Oct 2, 2018

Please see brave/brave-core#538 for the test plan, thanks.

@srirambv
Copy link
Contributor

srirambv commented Oct 4, 2018

Verification Passed on

Brave 0.55.11 Chromium: 70.0.3538.35 (Official Build) beta (64-bit)
Revision 28dcb499844fa40c28d5f62e337876cb936f79f5-refs/branch-heads/3538@{#678}
OS Linux
  • Verified WebRTC policy is listed under Privacy settings
  • Verified Default WebRTC setting shows IP as expected
  • Verified Default public and private interfaces setting shows IP as expected
  • Verified Disable non-proxied UDP blocks all IP as expected

Verified passed with

Brave 0.55.11 Chromium: 70.0.3538.35 (Official Build) beta(64-bit)
Revision 28dcb499844fa40c28d5f62e337876cb936f79f5-refs/branch-heads/3538@{#678}
OS Mac OS X

*Verified via test plan supplied in #551 (comment)

Verification passed on

Brave 0.55.11 Chromium: 70.0.3538.35 (Official Build) beta (64-bit)
Revision 28dcb499844fa40c28d5f62e337876cb936f79f5-refs/branch-heads/3538@{#678}
OS Windows 7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants