brave / brave-browser

Brave browser for Android, iOS, Linux, macOS, Windows.
https://brave.com
Mozilla Public License 2.0
17.06k stars 2.23k forks source link

settings cookie detail page displays differently for www.youtube.com vs youtube.com #37291

Closed LaurenWags closed 1 week ago

LaurenWags commented 3 months ago

Description

Encountered while testing https://github.com/brave/brave-browser/issues/37030

When trying to test the above issue, YouTube was still not displaying locally stored data when accessed via brave://settings/cookies/detail?site=www.youtube.com.

However, if we manually update this URL to brave://settings/cookies/detail?site=youtube.com, then data displays.

I'm not sure what can be done, as the steps below navigate to brave://settings/cookies/detail?site=www.youtube.com regardless of whether you visit www.youtube.com or just youtube.com. @emerick mentioned it might be due to how sites redirect via https://bravesoftware.slack.com/archives/CHGKGMHDJ/p1712151599129159?thread_ts=1711632360.658839&cid=CHGKGMHDJ.

Steps to Reproduce

  1. Clean profile, close and relaunch to pull griffin
  2. Open YouTube homepage or play any video on YouTube
  3. Click on tune icon in URL bar (to view site information)
  4. Click on Site settings
  5. Click on Cookies and Site Data
  6. Locally stored data is empty/blank

Actual result:

116a

Expected result:

116b

Reproduces how often:

easily

Brave version (brave://version info)

1.63.174, 1.64.116 (note, release versions in between these have a different issue, referenced above)

Version/Channel Information:

Other Additional Information:

Miscellaneous Information:

cc @rebron @brave/qa-team

GeetaSarvadnya commented 3 months ago

Reproduced the issue on

Brave | 1.64.116 Chromium: 123.0.6312.105 (Official Build) (64-bit)
-- | --
Revision | d9090cab16ea606108bdf6edd7466df58d9d76fd
OS | Windows 10 Version 22H2 (Build 19045.4170)

image

GeetaSarvadnya commented 3 months ago

This is a regression issue, it worked fine till 1.59.111 The issue started appearing from version 1.60.109

1.59.111 1.60.109
image image (1)
LaurenWags commented 1 week ago

Cookie detail page is being removed per https://github.com/brave/brave-browser/issues/39033, so this issue is no longer valid. Closing.