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

Add Amazon Publisher Services Tag surrogate; switch to extension URL-based surrogates #2801

Merged
merged 4 commits into from
Sep 23, 2021

Conversation

ghostwords
Copy link
Member

@ghostwords ghostwords commented Sep 17, 2021

  • Adds a surrogate for Amazon Publisher Services Tag as an extension URL, not a data URI string, to work around data URIs getting blocked by website CSPs
  • Converts all other surrogates to extension URLs

To be followed up by undoing 9869b70 once the update containing this PR gets widely distributed.

To test:

  1. Manually block c.amazon-adsystem.com
  2. Visit one of the affected CNN videos (example 1, example 2)
  3. Verify video works
  4. Comment out the surrogate pattern token entry for c.amazon-adsystem.com in hostnames to remove the redirect to our surrogate and make Privacy Badger simply block the domain
  5. Reload and verify the video is broken

Previously: #1411 (comment), #1793 (comment), #1858

@ghostwords ghostwords added the surrogates Replacement blank/noop/dummy/neutered versions of tracking scripts used to avoid site breakages label Sep 17, 2021
As a web accessible resource URL, not a data URI string, to work around
data URIs getting blocked by website CSPs.
@ghostwords ghostwords force-pushed the add-amazon-apstag-surrogate branch 2 times, most recently from 6770b50 to a1cd26e Compare September 21, 2021 17:04
@ghostwords ghostwords changed the title Add surrogate for Amazon Publisher Services Tag Add Amazon Publisher Services Tag surrogate and avoid surrogates getting blocked by CSPs Sep 21, 2021
@ghostwords ghostwords changed the title Add Amazon Publisher Services Tag surrogate and avoid surrogates getting blocked by CSPs Add Amazon Publisher Services Tag surrogate; switch to extension URL-based surrogates Sep 22, 2021
Broken by the update to manifest.json in 924f859
ghostwords added a commit that referenced this pull request Sep 23, 2021
Switch to extension URL-based surrogates to work around data URIs
getting blocked by website CSPs.

Add surrogate for Amazon Publisher Services Tag (to fix videos on CNN,
probably elsewhere).

To be followed by undoing 9869b70
@ghostwords ghostwords merged commit f534684 into master Sep 23, 2021
@ghostwords ghostwords deleted the add-amazon-apstag-surrogate branch September 23, 2021 16:10
ghostwords added a commit that referenced this pull request Sep 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
surrogates Replacement blank/noop/dummy/neutered versions of tracking scripts used to avoid site breakages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant