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

OS (Mac OS 13.2) DNS not honoured (or leaking) in Brave #28008

Open
nonokh opened this issue Jan 24, 2023 · 8 comments
Open

OS (Mac OS 13.2) DNS not honoured (or leaking) in Brave #28008

nonokh opened this issue Jan 24, 2023 · 8 comments

Comments

@nonokh
Copy link

nonokh commented Jan 24, 2023

Description

System-wide DNS setting (Adguard DoH). DNS leak in Brave browser only. No leaks in Safari.

Steps to Reproduce

  1. Go to https://www.dnsleaktest.com/
  2. Standard test

Actual result:

  1. Result in Brave

Screenshot 2023-01-24 at 9 50 47 PM

  1. Result in Safari

Screenshot 2023-01-24 at 9 52 27 PM

Expected result:

No dns leak

Reproduces how often:

Easily reproduced.

Brave version (brave://version info)

Brave | 1.46.153 Chromium: 108.0.5359.128 (Official Build) (arm64)
Revision | 1cd27afdb8e5d057070c0961e04c490d2aca1aa0-refs/branch-heads/5359@{#1185}
OS | macOS Version 13.2 (Build 22D49)

@nonokh nonokh changed the title DNS leak in Mac OS 13.2 OS (Mac OS 13.2) DNS not honoured (or leaking) in Brave Jan 24, 2023
@stephendonner
Copy link

Hi @nonokh - thanks for reporting, and sorry you're experiencing this issue. Can you help us reproduce this issue by providing as detailed step-by-step instructions as possible, here? Thanks!

Here's what I've tried, below.

Steps:

  1. on macOS, set DNS resolver for my home Wi-Fi to 94.140.14.14
  2. installed 1.49.64 Chromium: 110.0.5481.52 (Official Build) nightly (x86_64) on macOS Version 13.2
  3. confirmed AdGuard DNS resolver was used at the operating-system level, via nslookup and dig commands
  4. inbrave://settings/security I toggled Use Secure DNS to Off
  5. opened a new-tab page
  6. loaded www.dnsleaktest.com
  7. clicked on Standard Test
  8. confirmed detected DNS resolver domain was dns.adguard-dns.io., with two servers listed
example example example example example example example example example
Screenshot 2023-01-27 at 2 23 30 PM Screenshot 2023-01-27 at 2 27 06 PM Screenshot 2023-01-27 at 2 27 10 PM Screenshot 2023-01-27 at 2 27 00 PM Screenshot 2023-01-27 at 2 23 03 PM Screenshot 2023-01-27 at 2 28 51 PM Screenshot 2023-01-27 at 2 24 45 PM Screenshot 2023-01-27 at 2 25 04 PM Screenshot 2023-01-27 at 2 39 46 PM

@nonokh
Copy link
Author

nonokh commented Jan 28, 2023

Hi @stephendonner - I believe you misunderstood me. To reproduce my situation , you have to use the encrypted DoH (dns over https)config. I downloaded the config from their website here

The one you are using is their plain IPv4 dns server (the wifi section) and not encrypted dns. I am using the encrypted one for my system wide setting instead of the wifi dns way.

After you install the profile , doh will be available at (Settings -> Network -> VPN and filters -> Filters and proxy)
Screenshot 2023-01-28 at 5 15 31 PM

Now Safari uses the system setting and brave doesn't.

edit : fixed spellings

@stephendonner
Copy link

@nonokh ok, now using

Brave 1.49.75 Chromium: 110.0.5481.52 (Official Build) nightly (x86_64)
Revision 979113183ded4544a2c443aceb1629c430907e52-refs/branch-heads/5481@{#636}
OS macOS Version 13.2 (Build 22D49)

still can't reproduce. I always get adGuard.

What are your DNS settings in chrome://settings/security?

example example example example
Screenshot 2023-01-30 at 9 39 28 AM Screenshot 2023-01-30 at 9 39 36 AM Screenshot 2023-01-30 at 9 40 20 AM Screenshot 2023-01-30 at 9 40 46 AM

@frankebobdk
Copy link

@nonokh ok, now using

Brave | 1.49.75 Chromium: 110.0.5481.52 (Official Build) nightly (x86_64)

-- | --

Revision | 979113183ded4544a2c443aceb1629c430907e52-refs/branch-heads/5481@{#636}

OS | macOS Version 13.2 (Build 22D49)

still can't reproduce. I always get adGuard.

What are your DNS settings in chrome://settings/security?

example | example | example | example

---------|----------|---------|---------

Screenshot 2023-01-30 at 9 39 28 AM | Screenshot 2023-01-30 at 9 39 36 AM | Screenshot 2023-01-30 at 9 40 20 AM | Screenshot 2023-01-30 at 9 40 46 AM

Sorry to say, but look at your last picture - you're ain't on AdGuard dns server, but on NextDNS which ain't the same. So you just reproduced the issue yourself 😉

@stephendonner
Copy link

Sorry @nonokh and @frankebobdk yes, I can reproduce this now on latest nightly:

Screenshot 2023-01-30 at 11 18 41 AM

@rebron rebron added this to Untriaged Backlog in Security & Privacy via automation Jan 31, 2023
@rebron rebron added the needs-more-info The report requires more detail before we can decide what to do with this issue. label Jan 31, 2023
@rebron rebron added this to Untriaged Backlog in General Feb 1, 2023
@nonokh
Copy link
Author

nonokh commented Feb 3, 2023

@stephendonner glad you could reproduce.

Just for the heads up, the issue still persist (as of 3 Feb '23) in the latest official build too.

Current Brave version :

Brave 1.47.186 Chromium: 109.0.5414.119 (Official Build) (arm64)
Revision 772095164c7d5d4e73160f858efed3b5e87eca83-refs/branch-heads/5414@{#1458}
OS macOS Version 13.2 (Build 22D49)

recap tl;dr : Brave doesn't honour system DNS profile setting (which is different from the one in Wifi settings) even after turning Secure DNS off while Safari does.

@neeythann
Copy link

looks like this is a chromium issue @stephendonner - not sure if it's reported upstream.

Brave: Version 1.47.186 Chromium: 109.0.5414.119 (Official Build) (arm64)
Chrome: Version 109.0.5414.119 (Official Build) (arm64)
Safari: Version 16.1 (18614.2.9.1.12)

OS: macOS 13.0

image

@rebron rebron added repros-on-chrome and removed needs-more-info The report requires more detail before we can decide what to do with this issue. labels Feb 7, 2023
@stephendonner
Copy link

Yes, thanks @neeythann, this is indeed upstream Chromium issue: https://bugs.chromium.org/p/chromium/issues/detail?id=1378632

example example example
Screenshot 2023-02-02 at 1 22 18 AM Screenshot 2023-02-02 at 1 22 30 AM Screenshot 2023-02-02 at 1 22 43 AM

@ShivanKaul ShivanKaul added privacy/discussed Discussed in privacy confab Chromium/waiting upstream Issue is in Chromium; we'll likely wait for the fix labels Feb 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Untriaged Backlog
Security & Privacy
  
Untriaged Backlog
General
  
Untriaged Backlog
Development

No branches or pull requests

7 participants