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

Updated recommended Brave configuration #2561

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

5-tom
Copy link

@5-tom 5-tom commented May 1, 2024

Changes proposed in this PR:

  • I agree to the terms listed below:
    Contribution terms (click to expand) 1) I am the sole author of this work. 2) I agree to grant Privacy Guides a perpetual, worldwide, non-exclusive, transferable, royalty-free, irrevocable license with the right to sublicense such rights through multiple tiers of sublicensees, to reproduce, modify, display, perform, relicense, and distribute my contribution as part of this project. 3) I have disclosed any relevant conflicts of interest in my post. 4) I agree to the Community Code of Conduct.

…e users about Brave and Google Play Services
Copy link

github-actions bot commented May 1, 2024

Your preview is ready!

Name Link
🔨 Latest commit 4ca28fd
😎 Preview https://2561--glowing-salamander-8d7127.netlify.app/

@redoomed1
Copy link
Contributor

  • Warning about Brave's Google Play Services dependency

Please see the following update: https://discuss.privacyguides.net/t/latest-brave-for-android-has-a-hard-dependency-on-google-play-services-microg/18085/30

<div class="admonition warning" markdown>
<p class="admonition-title">Warning</p>

Brave for Android has a hard dependency on Google Play Services, and as a result will not work on [AOSP derivatives](https://www.privacyguides.org/en/android/#aosp-derivatives) without additional tweaks, such as [Sandboxed Google Play](https://grapheneos.org/features#sandboxed-google-play) on GrapheneOS.
Copy link
Contributor

@redoomed1 redoomed1 May 2, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is already a merged PR that fixes this regression (you can find it in the forum post I linked here), so this section shouldn't be necessary.

Copy link
Member

@jonaharagon jonaharagon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR!

As noted by @redoomed1, we don't want to jump the gun with adding warnings about issues that are ultimately temporary, unless they are long lasting or recurring problems. In this case it was an upstream issue that Brave is fixing.

Other changes LGTM at a glance, I'll check them out on my phone in a bit. Not sure about the safe browsing one. I think there was a previous discussion about the safe browsing setting already, but I'm not sure... I'll look for it.

docs/mobile-browsers.md Outdated Show resolved Hide resolved
Copy link
Contributor

@redoomed1 redoomed1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Regarding Safe Browsing on Brave (Android), here are some relevant excerpts and links to documentation:

  • Brave's privacy policy wrt SB

    Relevant excerpt

    On Android, we use the SafetyNet Safe Browsing API which sends partial URL hashes directly to Google when a URL is determined to be potentially malicious by the list stored locally on your device, as per the Safe Browsing Update API.

  • Brave's official documentation on their implementation of SB

    Relevant excerpts

    In the "Android" section

    Unlike the desktop browser (which downloads and maintains its own lists), the Android version of Brave makes use of a service provided by the operating system. Specifically, it uses the SafetyNet Google Play API and on-device lists that are shared between all of the applications performing Safe Browsing checks. Android devices without Google Play Services are currently unable to enable Safe Browsing in Brave.1

    Checking a URL against the local Safe Browsing lists follows the same steps as with the desktop browser, since the SafetyNet API also makes use of the Safe Browsing Update API. The only difference is that any requests from the operating system service to the Safe Browsing server (whether they originate from the Brave application or not) are done directly and do not go through a Brave proxy. This means that your IP address may be seen (and logged) by Google.

  • Developer documentation on the SafetyNet Safe Browsing API (referenced in the previous excerpts)

  • Developer documentation on the Safe Browsing Update API (referenced in Brave's Safe Browsing docs)

    Relevant excerpts

    The Update API lets your client applications download hashed versions of the Safe Browsing lists for storage in a local database. URLs can then be checked locally. Only if a match is found in the local database does the client need to send a request to the Safe Browsing servers to verify whether the URL is included on the Safe Browsing lists.

    In the "Checking URLs" section

    At no point does Google learn about the URLs you are examining. Google does learn the hash prefixes of URLs, but the hash prefixes don’t provide much information about the actual URLs.


This is all to say that the footnote under the optional suggestion about Safe Browsing should reference official docs, and offer more clarity about Brave's implementation of SB on specifically Android. I find the current footnote somewhat ambiguous.

I might draft one later if I have some time. See #2561 (comment).

Footnotes

  1. Emphasis mine

docs/mobile-browsers.md Outdated Show resolved Hide resolved
Co-authored-by: Jonah Aragon <jonah@triplebit.net>
Co-authored-by: redoomed1 <161974310+redoomed1@users.noreply.github.com>
Signed-off-by: 5-tom <132141431+5-tom@users.noreply.github.com>
@5-tom
Copy link
Author

5-tom commented May 7, 2024

Thank you for your changes! I've applied them to the file

Copy link
Member

@jonaharagon jonaharagon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this text different between iOS and Android, or what?

docs/mobile-browsers.md Outdated Show resolved Hide resolved
Signed-off-by: Jonah Aragon <jonah@triplebit.net>
Copy link
Member

@jonaharagon jonaharagon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR!

@jonaharagon jonaharagon requested a review from dngray May 23, 2024 05:52
@jonaharagon jonaharagon changed the title Updated recommended Brave configuration and new warning about Brave's Google Play Services dependency Updated recommended Brave configuration May 23, 2024
@jonaharagon jonaharagon added c:browsers browsers, add-ons, and related topics t:correction content corrections or errors labels May 23, 2024
Copy link
Contributor

@redoomed1 redoomed1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As is, LGTM.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c:browsers browsers, add-ons, and related topics t:correction content corrections or errors
Projects
Status: Needs Changes
Development

Successfully merging this pull request may close these issues.

None yet

3 participants