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

Remove "Gmail" language in SafeBrowsing text #16181

Closed
Miyayes opened this issue Jun 1, 2021 · 26 comments · Fixed by brave/brave-core#9401
Closed

Remove "Gmail" language in SafeBrowsing text #16181

Miyayes opened this issue Jun 1, 2021 · 26 comments · Fixed by brave/brave-core#9401
Assignees
Labels

Comments

@Miyayes
Copy link
Collaborator

Miyayes commented Jun 1, 2021

Description

Currently, some "Gmail" language appears in the browser.

image

Solution

Remove "Gmail" language in the Brave browser.

Reports

https://www.reddit.com/r/brave_browser/comments/nq2w0i/what_i_didnt_know_that_gmail_was_a_brave_service/

@srirambv srirambv added the OS/Android Fixes related to Android browser functionality label Jun 2, 2021
@srirambv
Copy link
Contributor

srirambv commented Jun 2, 2021

This is an issue on Android as well

@rebron rebron self-assigned this Jun 22, 2021
@rebron rebron added priority/P4 Planned work. We expect to get to it "soon". branding labels Jun 22, 2021
@Areckleh
Copy link

Reports through Brave Support Twitter DMs
image

@anthonypkeane anthonypkeane added priority/P3 The next thing for us to work on. It'll ride the trains. and removed priority/P4 Planned work. We expect to get to it "soon". labels Jun 28, 2021
@rebron
Copy link
Collaborator

rebron commented Jun 29, 2021

#11345 I think this is only Android and not on Desktop.

@rebron rebron removed their assignment Jun 29, 2021
@rebron rebron removed the OS/Desktop label Jun 29, 2021
@akshat-rawat
Copy link

Hi, I would like to work on this issue.
I have tracked down the root of this issue at brave-core/app/settings_strings.grdp, brave-core/browser/ui/android/strings/android_chrome_strings.grd and in .xtb files.

@srirambv
Copy link
Contributor

srirambv commented Jul 1, 2021

@akshat-rawat Assigned the issue to you. Please open a PR for it.

@akshat-rawat
Copy link

@srirambv, Thankyou for the opportunity, I just wanted to confirm the changes.
From.
Does not protect you against dangerous websites, downloads and extensions. You’ll still get Safe Browsing protection, where available, in other Brave services, like Gmail and Search.
To,
Does not protect you against dangerous websites, downloads and extensions.

Or is there a better alternative?

@srirambv
Copy link
Contributor

srirambv commented Jul 1, 2021

Looping in @fmarier @karenkliu for text change. We may want to keep the You'll still get Safe Browsing protection part

@akshat-rawat
Copy link

What about?
Does not protect you against dangerous websites, downloads and extensions. You’ll still get Safe Browsing protection, where available, in other Brave services, like Search and Passwords.

@karenkliu
Copy link

cc @rmcfadden3 on this for the text change. We have an in-house copywriter now and should filter all our text through them.

@akshat-rawat
Copy link

@karenkliu, please let me know if there are any updates on what the changed text should be.

@rmcfadden3
Copy link

How about this:

Does not protect you against dangerous websites, downloads, or extensions. (Where available, you’ll still get safe browsing protection in other Brave services.)

@srirambv
Copy link
Contributor

srirambv commented Jul 2, 2021

@rmcfadden3 We proxy safe browsing requests so we may want to somehow mention that in there. Someone from @brave/sec-team may pitch in on it.

@soham4abc
Copy link

soham4abc commented Jul 2, 2021

I am working on this issue. making the message " Does not protect you against dangerous websites, downloads, and extensions. (Where available, you’ll still get safe browsing protection in other Brave services.)"

@srirambv is it okay?

@srirambv
Copy link
Contributor

srirambv commented Jul 2, 2021

@soham4abc Think @akshat-rawat may already have a PR ready for it. You can always choose any issues from here.

As for the wording I'd wait till someone from security team responds and @rmcfadden3 can update accordingly which can be used.

@soham4abc
Copy link

@soham4abc Think @akshat-rawat may already have a PR ready for it. You can always choose any issues from here.

As for the wording I'd wait till someone from security team responds and @rmcfadden3 can update accordingly which can be used.

Okay then I am making a PR. If the wording needs to be changed I will do so.
If possible please do assign me for this issue!
Thank you!

@soham4abc
Copy link

@soham4abc Think @akshat-rawat may already have a PR ready for it. You can always choose any issues from here.

As for the wording I'd wait till someone from security team responds and @rmcfadden3 can update accordingly which can be used.

Okay sure! I already made a PR and if required will close it if @akshat-rawat has the PR I will close my PR later.

@akshat-rawat
Copy link

@soham4abc I do have a PR ready just wanted to confirm the changed text.

@soham4abc
Copy link

@soham4abc I do have a PR ready just wanted to confirm the changed text.

Sure... My PR is closed! Go ahead!

@akshat-rawat
Copy link

Is "Does not protect you against dangerous websites, downloads, or extensions. (Where available, you’ll still get safe browsing protection in other Brave services.)"
final or are we still waiting, @srirambv?

@srirambv
Copy link
Contributor

srirambv commented Jul 2, 2021

We should wait on #16181 (comment)

@fmarier
Copy link
Member

fmarier commented Jul 2, 2021

Given that we don't run GMail and Google Search, I think we should simply drop the second sentence and just have:

Does not protect you against dangerous websites, downloads, or extensions.

That's why it's not recommended. You're turning off all of the Google Safe Browsing protections if you choose that option.

@akshat-rawat
Copy link

Should i move forward with Does not protect you against dangerous websites, downloads, or extensions.?

@srirambv
Copy link
Contributor

srirambv commented Jul 5, 2021

@akshat-rawat yes please. Go ahead and create the PR

@akshat-rawat
Copy link

@srirambv, I have created the PR.
If there are any updates required, do let me know.

@bsclifton
Copy link
Member

Removing assignment as above PR was closed

@srirambv
Copy link
Contributor

srirambv commented Aug 2, 2021

Verification passed on the following devices running 1.28.99 64 Beta build

  • Verified references to Brave is removed from safe browsing settings
Oppo R5 (Android 11) Samsung Tab A (Android 10)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment