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

Attempting to delete brave shields generated cookie settings can sometimes create unrecoverable inconsistencies between the shield setting and the actual cookie setting #20858

Closed
bridiver opened this issue Feb 3, 2022 · 1 comment · Fixed by brave/brave-core#12116
Assignees
Labels
OS/Android Fixes related to Android browser functionality OS/Desktop QA/No release-notes/exclude

Comments

@bridiver
Copy link
Contributor

bridiver commented Feb 3, 2022

Description

Attempting to delete brave shields generated cookie settings can sometimes create unrecoverable inconsistencies between the shield setting and the actual cookie setting

Steps to Reproduce

  1. go to brave.com
  2. toggle shields cookie settings to block all and back to block cross-site again
  3. disable shields for brave.com
  4. go to brave://settings/cookies and click the trash can icon for the rule allowing embedded cookies on brave.com
  5. enable shields for brave.com

Actual result:

Shields says cross site cookies are block, but brave://settings/cookies says all cookies are allowed and that is likely the actual setting as well

Expected result:

Cross site cookies should still be blocked

Reproduces how often:

Always

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

1.36.39

Version/Channel Information:

all channels

  • Can you reproduce this issue with the current release? yes
  • Can you reproduce this issue with the beta channel? yes
  • Can you reproduce this issue with the dev channel? yes
  • Can you reproduce this issue with the nightly channel? yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? no
  • Does the issue resolve itself when disabling Brave Rewards? no
  • Is the issue reproducible on the latest version of Chrome? NA
@bridiver
Copy link
Contributor Author

bridiver commented Feb 4, 2022

I left this as no QA because this functionality was already broken, this just prevents it from causing more problems for people until we have a real fix

@bridiver bridiver added OS/Android Fixes related to Android browser functionality OS/Desktop labels Feb 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OS/Android Fixes related to Android browser functionality OS/Desktop QA/No release-notes/exclude
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant