Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 29349794: Issue 4280 - Begin whitelisting domains used by Google redirects (Closed)

Created:
Aug. 12, 2016, 4:58 p.m. by kzar
Modified:
Aug. 12, 2016, 5:12 p.m.
Reviewers:
Sebastian Noack
Visibility:
Public.

Description

Issue 4280 - Begin whitelisting domains used by Google redirects

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -1 line) Patch
M easylist+exceptionrules_content_blocker.json View 1 chunk +2 lines, -1 line 1 comment Download

Messages

Total messages: 2
kzar
Patch Set 1
Aug. 12, 2016, 4:59 p.m. (2016-08-12 16:59:53 UTC) #1
Sebastian Noack
Aug. 12, 2016, 5:10 p.m. (2016-08-12 17:10:04 UTC) #2
https://codereview.adblockplus.org/29349794/diff/29349795/easylist+exceptionr...
File easylist+exceptionrules_content_blocker.json (right):

https://codereview.adblockplus.org/29349794/diff/29349795/easylist+exceptionr...
easylist+exceptionrules_content_blocker.json:655857: +				"bs.serving-sys.com"
LGTM as quick fix. But note that this won't work if the redirect goes to a
different domain, and this not being canonical Google URL, I could imagine that
it's not always the same.

Powered by Google App Engine
This is Rietveld