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

Brave shields always down by default on sites, but enabled in global settings #24514

Closed
FearNaBoinne opened this issue Aug 8, 2022 · 18 comments · Fixed by brave/brave-core#14592

Comments

@FearNaBoinne
Copy link

Description

Despite being enabled in global settings, any new site loads with shields down, and I have to manually enable them!

Steps to Reproduce

  1. Enable shields in global settings
  2. Load new site
  3. See that the icon is grey and shields are down

Actual result:

Shields down

Expected result:

Shields up

Reproduces how often:

100%

Brave version (info found on brave://version)

Brave | 1.42.86 Chromium: 104.0.5112.81 (Official Build) (64-bit)
Revision | 5b7b76419d50f583022568b6764b630f6ddc9208-refs/branch-heads/5112@{#1309}
OS | Windows 10 Version 21H2 (Build 19044.1826)
JavaScript | V8 10.4.132.20
User Agent | Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/104.0.5112.81 Safari/537.36

Version/Channel Information:

  • Can you reproduce this issue with the current release?
    I am on current
  • Can you reproduce this issue with the beta channel?
    I haven't tried, but it's been several versions now
  • Can you reproduce this issue with the dev channel?
    I haven't tried, but it's been several versions now
  • Can you reproduce this issue with the nightly channel?
    I haven't tried, but it's been several versions now
@FearNaBoinne
Copy link
Author

2022-08-08 10_29_21-Settings - Brave

@WalletUpLLC
Copy link

Experiencing the same exact issues for several weeks now.
Version 1.42.86 Chromium: 104.0.5112.81 (Official Build) (x86_64 on Macbook Pro 11.6.8 (20G730)

@diracdeltas diracdeltas added the feature/shields The overall Shields feature in Brave. label Aug 9, 2022
@rebron rebron added needs-investigation A bug not 100% confirmed/fixed needs-more-info The report requires more detail before we can decide what to do with this issue. OS/Desktop labels Aug 9, 2022
@rebron
Copy link
Collaborator

rebron commented Aug 9, 2022

@FearNaBoinne and @WalletUpLLC Can you post a screenshot of the settings you have in brave://settings/cookies? Don't need to include the customize behaviors section.

@boocmp
Copy link

boocmp commented Aug 10, 2022

@FearNaBoinne and @WalletUpLLC do you use Sync?

@WalletUpLLC
Copy link

@FearNaBoinne and @WalletUpLLC Can you post a screenshot of the settings you have in brave://settings/cookies? Don't need to include the customize behaviors section.

Screen Shot 2022-08-09 at 11 58 44 PM

Please find in attached. It has been working for several months , till the several past weeks ago. Did not make report because assumed it was a glitch that would be resolved till now.

@WalletUpLLC
Copy link

@FearNaBoinne and @WalletUpLLC do you use Sync?

it happens with or without

@FearNaBoinne
Copy link
Author

@FearNaBoinne and @WalletUpLLC Can you post a screenshot of the settings you have in brave://settings/cookies? Don't need to include the customize behaviors section.

Screen Shot 2022-08-09 at 11 58 44 PM

Please find in attached. It has been working for several months , till the several past weeks ago. Did not make report because assumed it was a glitch that would be resolved till now.

Mine are identical to @WalletUpLLC

@FearNaBoinne and @WalletUpLLC do you use Sync?

it happens with or without

Same as well, but right now sync is enabled...

@boocmp
Copy link

boocmp commented Aug 10, 2022

@FearNaBoinne Could you please open brave://sync-internals, go to the "Sync Node Browser" tab, expand Preferences and find encrypted node with name:"profile.content_settings.exceptions.braveShields". Does it contain a rule with the key "*,*"?
image

@FearNaBoinne
Copy link
Author

FearNaBoinne commented Aug 10, 2022

@FearNaBoinne Could you please open brave://sync-internals, go to the "Sync Node Browser" tab, expand Preferences and find encrypted node with name:"profile.content_settings.exceptions.braveShields". Does it contain a rule with the key "*,*"? image

There is NO encrypted node with that name, only a normal named node!
(I only have 4 encrypted nodes, "extensions.pinned_extensions", "profile.content_settings.exceptions.popups", "extensions.commands" and "profile.content_settings.exceptions.font_access")

The normal named node DOES have a "*,*":
2022-08-10 11_36_56-Sync Internals - Brave

@boocmp
Copy link

boocmp commented Aug 10, 2022

Thanks a lot! I guess we have found the cause of the problem

@FearNaBoinne
Copy link
Author

Thanks a lot! I guess we have found the cause of the problem

And how do we fix it? :P

@boocmp
Copy link

boocmp commented Aug 10, 2022

Uhhh, I guess disabling synchronization and restarting the browser should fix this. But I'm afraid there is no workaround if you want to use synchronization. I believe we will release a fix in near future.

@FearNaBoinne
Copy link
Author

Then I will wait for that fix to come around! Thx!

@WalletUpLLC
Copy link

Uhhh, I guess disabling synchronization and restarting the browser should fix this. But I'm afraid there is no workaround if you want to use synchronization. I believe we will release a fix in near future.

same in here, but got 38 encrypted, and {"," found in an unencrypted profile.content_settings.exceptions.braveShields

However there is a partial workaround.
Since the issue is narrowed down to the "settings preferences", and assuming that the settings were already sync across the synced devices, you can safely go to brave://settings/braveSync/setup and select the "customize sync" target and disable "settings". Restart brave and you should be good to go. You want to apply same process on the profiles where you want it to load global shield settings
Screen Shot 2022-08-10 at 9 08 09 AM

@FearNaBoinne try and see how it works for you

@brave-builds brave-builds added this to the 1.44.x - Nightly milestone Aug 13, 2022
@rebron rebron removed needs-investigation A bug not 100% confirmed/fixed needs-more-info The report requires more detail before we can decide what to do with this issue. labels Aug 17, 2022
@kjozwiak
Copy link
Member

@brave/qa-team added some more detailed/easier to follow STR/Cases via brave/brave-core#14592 (comment) based on the information from @boocmp via brave/brave-core#14592 (comment).

@kjozwiak
Copy link
Member

The above will require 1.43.80 or higher for 1.43.x verification 👍

@stephendonner stephendonner changed the title Brave shields always down be default on sites, but enabled in global settings Brave shields always down by default on sites, but enabled in global settings Aug 23, 2022
@stephendonner
Copy link

stephendonner commented Aug 23, 2022

Verification PASSED using

Brave 1.43.80 Chromium: 104.0.5112.102 (Official Build) dev (x86_64)
Revision 8e5396254975ef939f2ef7d0bd334e48a052b536-refs/branch-heads/5112@{#1478}
OS macOS Version 11.6.8 (Build 20G730)

Reproduced the issue on macOS x64 using the following STR/Cases and build(s):

{"*,*":{"expiration":"0","last_modified":"13304670271801570","model":0,"setting":2}}
  • launched 1.40.113 Chromium: 103.0.5060.114 and ensured that shields were disabled on all websites visited
  • enabled sync chain via brave://settings/braveSync and enabled Sync everything (waited ~2-3mins)
  • closed 1.40.113 Chromium: 103.0.5060.114
  • installed 1.43.80 Chromium: 104.0.5112.81 and joined the sync chain that was created with 1.40.113 (using the daily code)
  • enabled Sync everything (waited ~2-3mins)

Once the above was done, any website visited within 1.40.113 also had shields disabled. Examples below re: issue occurring:

Issue occurring via 1.40.113 Chromium: 103.0.5060.114

Example Example Example
Screen Shot 2022-08-23 at 1 23 59 PM Screen Shot 2022-08-23 at 1 24 07 PM Screen Shot 2022-08-23 at 1 24 27 PM

Verification PASSED on macOS-x64 using the following build(s):

Brave | 1.43.80 Chromium: 104.0.5112.102 (Official Build) dev (x86_64)
-- | --
Revision | 8e5396254975ef939f2ef7d0bd334e48a052b536-refs/branch-heads/5112@{#1478}
OS | macOS Version 11.6.8 (Build 20G730)

Using the STR/Cases mentioned above, went through the same process but using 1.43.80 Chromium: 104.0.5112.102 and ensured that shields were working as expected, per the following:

Issue occurring via 1.40.113 Chromium: 103.0.5060.114

Example Example Example
Screen Shot 2022-08-23 at 4 15 18 PM Screen Shot 2022-08-23 at 4 15 34 PM Screen Shot 2022-08-23 at 4 16 08 PM

Shields working as expected via 1.43.80 Chromium: 104.0.5112.102

Example Example Example
Screen Shot 2022-08-23 at 4 16 48 PM Screen Shot 2022-08-23 at 4 17 02 PM Screen Shot 2022-08-23 at 4 19 22 PM

Once both versions synced, ensured that shields were also working via 1.40.113 Chromium: 103.0.5060.114.

Screen Shot 2022-08-23 at 4 25 44 PM

@stephendonner stephendonner added QA/In-Progress Indicates that QA is currently in progress for that particular issue QA Pass-macOS and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Aug 23, 2022
@MadhaviSeelam
Copy link

MadhaviSeelam commented Aug 25, 2022

Verification PASSED using

Brave | 1.43.81 Chromium: 104.0.5112.102 (Official Build) (64-bit)
-- | --
Revision | 8e5396254975ef939f2ef7d0bd334e48a052b536-refs/branch-heads/5112@{#1478}
OS | Windows 11 Version 21H2 (Build 22000.856)

Reproduced the issue using the following STR/Cases and build(s):

  • Installed 1.40.113 Chromium: 103.0.5060.114
  • launched Brave
  • visited https://cnn.com and disable Shields
  • closed 1.40.113 Chromium: 103.0.5060.114
  • opened the preferences file via \AppData\Local\BraveSoftware\Brave-Browser\User Data\Default
  • edited the profile.content_settings.exceptions.braveShields key as per the following:
{"*,*": {"expiration":"0","last_modified":"13305939022714591","model":0,"setting":2})
  • launched 1.40.113 Chromium: 103.0.5060.114 and ensured that shields were disabled on all websites visited
  • enabled sync chain via brave://settings/braveSync and enabled Sync everything (waited ~2-3mins)
  • closed 1.40.113 Chromium: 103.0.5060.114
  • installed 1.43.81 Chromium: 104.0.5112.81 and joined the sync chain that was created with 1.40.113 (using the daily code)
  • enabled Sync everything (waited ~2-3mins)

Once the above was done, any website visited within 1.40.113 also had shields disabled. Examples below re: issue occurring:

Issue occurring via 1.40.113 Chromium: 103.0.5060.114

Ex1 Ex2 Ex3 Ex4
image image image image

Verification on a new profile 1.43.81 Chromium: 104.0.5112.102

  • Installed 1.43.81
  • launch Brave
  • visited https:// cnn.com and disabled shields
  • enabled sync chain via brave://settings/braveSync and enabled Sync everything (waited ~2-3mins)
  • close the browser
  • opened the preferences file via \AppData\Local\BraveSoftware\Brave-Browser\User Data\Default
  • edited the profile.content_settings.exceptions.braveShields key as per the following:
    {"*,*": {"expiration":"0","last_modified":"13305939745950080","model":0,"setting":2})
    
  • relaunch Brave
  • confirmed Shields is enabled and working as expected.
  • visited other sites and confirmed Shields are enabled and working as expected (nytimes.com) and (bbc.com)
Ex1 Ex2 Ex3 Ex4
image image image image

Verification passed on

Brave 1.43.81 Chromium: 104.0.5112.102 (Official Build) (64-bit)
Revision 8e5396254975ef939f2ef7d0bd334e48a052b536-refs/branch-heads/5112@{#1478}
OS Ubuntu 18.04 LTS

Reproduced the issue using the following STR/Cases and build(s):

  • Installed 1.40.113 Chromium: 103.0.5060.114
  • launched Brave
  • visited https://cnn.com and disable Shields
  • closed 1.40.113 Chromium: 103.0.5060.114
  • opened the preferences file
  • edited the profile.content_settings.exceptions.braveShields key as per the following:
{"*,*": {"expiration":"0","last_modified":"13305939022714591","model":0,"setting":2})
  • launched 1.40.113 Chromium: 103.0.5060.114 and ensured that shields were disabled on all websites visited
  • enabled sync chain via brave://settings/braveSync and enabled Sync everything (waited ~2-3mins)
  • closed 1.40.113 Chromium: 103.0.5060.114
  • installed 1.43.81 Chromium: 104.0.5112.81 and joined the sync chain that was created with 1.40.113 (using the daily code)
  • enabled Sync everything (waited ~2-3mins)

Once the above was done, any website visited within 1.40.113 also had shields disabled. Examples below re: issue occurring:

Issue occurring via 1.40.113 Chromium: 103.0.5060.114

Verification on a new profile 1.43.81 Chromium: 104.0.5112.102

Verified the above test plan but with 1.43.81

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