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

Clearing Shields Per-Site Settings #3926

Open
tildelowengrimm opened this issue Mar 28, 2019 · 2 comments
Open

Clearing Shields Per-Site Settings #3926

tildelowengrimm opened this issue Mar 28, 2019 · 2 comments
Labels
feature/global-settings Settings at browser level independent of shields settings feature/shields The overall Shields feature in Brave. priority/P3 The next thing for us to work on. It'll ride the trains.

Comments

@tildelowengrimm
Copy link
Contributor

None of the options in the "Clear Browsing Data" settings item explicitly call out Shields, but certainly some of them affect Shields, because I just restarted my browser with some of them selected and my per-site and global Shields settings were reset.

We should specifically call out Shields settings in the clear-on-exit menu.

@tildelowengrimm tildelowengrimm added feature/shields The overall Shields feature in Brave. feature/global-settings Settings at browser level independent of shields settings priority/P3 The next thing for us to work on. It'll ride the trains. labels Mar 28, 2019
@sbtnc
Copy link

sbtnc commented Jun 4, 2019

Yes, indeed. When "Site Settings" is checked in "Clear browsing data", it causes a reset on some (not all) shields settings.

Maybe "Site Settings" label is misleading (as I would expect specifics settings not global ones) or it's a bug.

For my case, I have a privacy oriented profile for daily surfing (whitelisting approach). Thus clearing all browsing data on exit does make sense. However, defaults shields settings get reseted on restart which are least private settings.

Steps to Reproduce

  1. In "Settings" > "Brave shields default" set to non defaults "Ad Control", "Cookie Control" and "Fingerprinting protection";
  2. In "Settings" > "Clear browsing data" > "On exit" check "Site Settings";
  3. Restart Brave Browser;

Actual result:

Global settings are back to defaults.

Expected result:

Global settings don't get back to defaults.

Reproduces how often:

Always.

Brave version

Brave 0.64.77 Chromium: 74.0.3729.169 (Official Build) (64-bit)
Revision 78e4f8db3ce38f6c26cf56eed7ae9b331fc67ada-refs/branch-heads/3729@{#1013}
OS Mac OS X

@rebron
Copy link
Collaborator

rebron commented Apr 29, 2020

@tomlowenthal Fixed with #8826?

@rebron rebron closed this as completed Apr 29, 2020
@rebron rebron reopened this Apr 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/global-settings Settings at browser level independent of shields settings feature/shields The overall Shields feature in Brave. priority/P3 The next thing for us to work on. It'll ride the trains.
Projects
Status: P3 backlog
Development

No branches or pull requests

3 participants