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

global shields setting for blocking all cookies isn’t being applied #6389

Closed
LaurenWags opened this issue Oct 9, 2019 · 4 comments · Fixed by brave/brave-core#3694
Closed
Labels
feature/shields/cookies Cookie controls implemented as part of Shields. priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Test-Plan-Specified QA/Yes release-notes/include

Comments

@LaurenWags
Copy link
Member

Description

Found while testing #5740

If you change your default shield setting for cookies to be Block all cookies it does not work. Shields show Cookies Blocked but not all cookies are being blocked.

Note - default value of Only block cross-site cookies and Allow all cookies both work as the global setting.

This was working with 0.68.132 (last released C76 version). It appears to be broken with 0.68.138 (first released C77 version) and later versions.

Steps to Reproduce

  1. Install a C77 version (0.68.138 on release channel or newer)
  2. Go to settings, change default shield setting to 'Block all Cookies'
  3. Open a new tab, go to http://www.whatarecookies.com/cookietest.asp
  4. Page says cookies are Enabled.
  5. Open shields, shields shows 'Cookies Blocked'
  6. Change to 'Cross-site cookies blocked'
  7. Page refreshes, still shows cookies Enabled.
  8. Open shields, change back to 'Cookies Blocked'
  9. Now page shows cookies are Disabled.

Actual result:

Step 4, page shows cookies Enabled
Screen Shot 2019-10-09 at 4 17 31 PM

If I click on 'Not Secure' in URL bar, it shows 2 cookies:
Screen Shot 2019-10-09 at 4 23 35 PM

Expected result:

Page should show cookies are Disabled.
Screen Shot 2019-10-09 at 4 18 26 PM

Screen Shot 2019-10-09 at 4 24 50 PM

Reproduces how often:

easily

Brave version (brave://version info)

reproduces on 0.68.138

also reproduces with

Brave 0.69.132 Chromium: 77.0.3865.90 (Official Build) (64-bit)
Revision 58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830}
OS macOS Version 10.13.6 (Build 17G5019)
Brave 0.70.111 Chromium: 77.0.3865.90 (Official Build) beta (64-bit)
Revision 58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830}
OS macOS Version 10.13.6 (Build 17G5019)
Brave 0.72.83 Chromium: 77.0.3865.90 (Official Build) nightly (64-bit)
Revision 58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830}
OS macOS Version 10.13.6 (Build 17G5019)

Version/Channel Information:

  • 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? unsure
  • Can you reproduce this issue with the nightly channel? yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? n/a
  • Does the issue resolve itself when disabling Brave Rewards? n/a
  • Is the issue reproducible on the latest version of Chrome? n/a

Miscellaneous Information:

cc @brave/legacy_qa @rebron

@LaurenWags LaurenWags added QA/Yes QA/Test-Plan-Specified feature/shields/cookies Cookie controls implemented as part of Shields. labels Oct 9, 2019
@LaurenWags LaurenWags changed the title changing default shield setting to Block all Cookies does not work changing global default shield setting to Block all Cookies does not work Oct 9, 2019
@btlechowski
Copy link

Reproduced on

Brave 0.70.111 Chromium: 77.0.3865.90 (Official Build) beta (64-bit)
Revision 58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830}
OS Ubuntu 18.04 LTS

@LaurenWags LaurenWags changed the title changing global default shield setting to Block all Cookies does not work global shields setting for blocking all cookies isn’t being applied Oct 9, 2019
@rebron rebron added the priority/P2 A bad problem. We might uplift this to the next planned release. label Oct 9, 2019
@bsclifton
Copy link
Member

bsclifton commented Oct 9, 2019

Root cause / fix might be similar to #5973 / brave/brave-core#3405
cc: @bridiver

@GeetaSarvadnya
Copy link

Reproduced on

Brave 0.70.112 Chromium: 77.0.3865.90 (Official Build) beta (64-bit)
Revision 58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830}
OS Windows 10 OS Version 1803 (Build 17134.1006)

@LaurenWags
Copy link
Member Author

LaurenWags commented Oct 17, 2019

Verified passed with

Brave 0.70.118 Chromium: 78.0.3904.50 (Official Build) (64-bit)
Revision 2accdc52c79976e264cd2694df6db31d1fccd8e8-refs/branch-heads/3904@{#658}
OS macOS Version 10.13.6 (Build 17G5019)
  • Verified STR from description, on step 4 page showed cookies as 'Disabled' as expected.

Screen Shot 2019-10-17 at 12 20 58 PM

Screen Shot 2019-10-17 at 12 20 40 PM

Verification passed on

Brave 0.70.118 Chromium: 78.0.3904.50 (Official Build) (64-bit)
Revision 2accdc52c79976e264cd2694df6db31d1fccd8e8-refs/branch-heads/3904@{#658}
OS Ubuntu 18.04 LTS

Verified STR from the description

image

Verification passed on

Brave 0.70.118 Chromium: 78.0.3904.50 (Official Build) (64-bit)
Revision 2accdc52c79976e264cd2694df6db31d1fccd8e8-refs/branch-heads/3904@{#658}
OS Windows 10 OS Version 1803 (Build 17134.1006)

-Verified STR from the description
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/shields/cookies Cookie controls implemented as part of Shields. priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Test-Plan-Specified QA/Yes release-notes/include
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants