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

drive.google.com #396

Closed
GRadziejewski opened this issue Nov 30, 2023 · 4 comments
Closed

drive.google.com #396

GRadziejewski opened this issue Nov 30, 2023 · 4 comments
Labels
reproduction/success This issue can be reproduced type/anti-tracking Issues with Anti-Tracking

Comments

@GRadziejewski
Copy link
Contributor

Description:
Selected more than 2 files that download as a ZIP file are ultimately not downloaded

Steps to Reproduce:

  1. Login into gmail account
  2. Open https://drive.google.com/drive/home
  3. Select more than 2 files
  4. Click right mouse button to open menu and select download
  5. Wait for the creation of ZIP file and automatically downloading (12 sec on the video below)
Video.2023-11-30.10-27-50-196.mp4

Expected Results:
ZIP files should be downloaded normally

Versions:

Browser: Firefox 120.0
OS: Windows 10 Pro 
GBE 8.11.1

Browser: Chrome 119.0.6045.125
OS: Windows 10 Pro 
GBE 8.12.4

Browser: Opera One 105.0.4970.21
OS: Windows 10 Pro
GBE 8.12.4

Browser: Edge 119.0.2151.97 
OS: Windows 10 Pro 
GBE 8.12.4
@chrmod
Copy link
Member

chrmod commented Dec 12, 2023

The breakage starts here - https://github.com/ghostery/common/blob/e35151f9222c514be8340161aa7cdcc63a619710/modules/antitracking/sources/attrack.es#L685

Question is why Anti-Tracking decided it is unsafe cookie.

@chrmod
Copy link
Member

chrmod commented Dec 12, 2023

Looks like Anti-Tracking removes auth cookies for googleusercontent.com

Screenshot 2023-12-12 at 10 07 22

@philipp-classen
Copy link
Member

philipp-classen commented Dec 12, 2023

Anti-tracking prevents Google from setting an essential cookie. I updated https://cdn.ghostery.com/antitracking/config.json to include an exception:

compatibilityList: { "googleusercontent.com": ["google.com"], ... }

However, the compatibility list is currently broken until we following code change is released as well: ghostery/common#95

@seia-soto
Copy link
Member

Cannot reproduce. Assuming this issue is already fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
reproduction/success This issue can be reproduced type/anti-tracking Issues with Anti-Tracking
Projects
None yet
Development

No branches or pull requests

4 participants