-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Provide a Brave-specific User Agent for some sites #1052
Comments
As a general policy, we don't want the |
FWIW, a couple of cases where a distinctive string might be problematic:
Related, in the Waterfox area: |
From the results I see for the browser - https://panopticlick.eff.org/results?aat=1#fingerprintTable The UserAgent string displays the following:
A simpler way to customize this User Agent string to mask this information could be a starting point for customization. |
Updated the title. We feel like we have a big enough user base now to go ahead with adding Brave to the end of the User-Agent. So instead of: We'd show: |
@bbondy for a slew of historical reasons the E.g. |
Thanks @da2x we were considering just doing Brave/1.0 in a side discussion but maybe the Chrome version is better. |
Using +1 for |
@bbondy More reasons to copy the Chr version: HTTP/2 uses header compression and repeated [version] strings compress better than unique ones. 😋 It also doesn't leak any additional data. @Fornax96 Have a look at the never ending battle over this in the WebCompat database. It's not that easy. That being said, Brave could probably go with just |
Updated description
There should be a way to share a Brave-specific User Agent
We currently do this with our "site hacks":
https://github.com/brave/brave-core/blob/66ceaf3491778f7d9a62f215c8fb47b318c9b181/browser/net/brave_site_hacks_network_delegate_helper.cc#L62-L76
User Agent example format:
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_2) AppleWebKit/537.36 (KHTML, like Gecko) Brave Chrome/79.0.3945.117 Safari/537.36
This user agent is only shared with a few hard-coded sites, current list found here:
https://github.com/brave/brave-core/blob/4621aa3f71758f3bd890f381f5774d91a0338b2e/common/shield_exceptions.cc#L17-L29
The scope for this issue would be to expand to our list to be dynamic (remove hard-coded entries) and match the list we provide for our supported publishers (already fetched on browser launch).
https://laptop-updates.brave.com/promo/custom-headers
Original issue description
Carried over from brave/browser-laptop#3693
Related #4641
We may need these initial overrides to start:
UA overrides in Edge: edge://compat/useragent
The text was updated successfully, but these errors were encountered: