brave/brave-browser

Probe for Brave-owned domains instead of Google ones when user selects custom DoH provider

Open

#38 555 ouverte le 24 mai 2024

Voir sur GitHub
 (5 commentaires) (1 réaction) (0 assignés)JavaScript (2 164 forks)batch import
OS/AndroidOS/Desktopgood first issuepriority/P5privacy/connect

Métriques du dépôt

Stars
 (16 194 stars)
Métriques de merge PR
 (Merge moyen 4h 38m) (19 PRs mergées en 30 j)

Description

Description

As discussed in https://github.com/brave/brave-browser/issues/38058, when the user chooses a custom DoH provider, Brave asks the new DoH provider to resolve gstatic.com and google.com. Importantly, the connection is not actually made to these domains, it's simply a test query that tells the DoH provider that some Chromium-based browser has selected it to be its custom DoH provider. This is not a privacy leak, but it might be worth patching out just for appearance sake ("why is Brave doing a DNS query for google domains?")

See: https://github.com/GrapheneOS/Vanadium/blob/main/patches/0064-use-grapheneos.network-when-verifying-DNS-over-HTTPS.patch

Steps to reproduce

Follow the steps on https://github.com/brave/brave-browser/issues/38058#issuecomment-2118199829

Actual result

DoH query for google and gstatic

Expected result

DoH query for a Brave-owned domain

Reproduces how often

Easily reproduced

Desktop Brave version (brave://version info)

All

Android device

All

Channel information

  • release (stable)
  • beta
  • nightly

Reproducibility

  • with Brave Shields disabled
  • with Brave Rewards disabled
  • in the latest version of Chrome

Miscellaneous information

No response

Guide contributeur

Direction de recherche
Inspectez le patch GrapheneOS et adaptez le pour Brave, en remplaçant les domaines Google par des domaines appartenant à Brave. Testez les modifications.
Stack technique
javascriptc
Domaine
networkingsecurity
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70