Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

whitelist script signatures #1353

Closed
bridiver opened this issue Apr 13, 2016 · 7 comments
Closed

whitelist script signatures #1353

bridiver opened this issue Apr 13, 2016 · 7 comments

Comments

@bridiver
Copy link
Collaborator

bridiver commented Apr 13, 2016

Test Plan

  1. Go to adops.com w/ shields on
  2. Ensure the site is usable
  3. Go to shields hit ads and trackers blocked
  4. List of blocked ads should not include the first-party domain

I recently went to adops.com and the site was basically unusable because they are on the blocking list. I don't really see any reason why we should be blocking jquery and other scripts that are not ad related. Not sure if a whitelist is the answer, but maybe consider the referrer url (when not in an iframe) because these are 1st-party scripts in this context.

@diracdeltas
Copy link
Member

this is a bug - TP/AB should not be blocking first party resources in the first place

@diracdeltas
Copy link
Member

need @bbondy input. i would have expected adblock.matches(details.url, mapFilterType[details.resourceType], firstPartyUrl.host) to return false when details.url is first-party to firstPartyUrl.host. apparently this is not the case. should this be fixed in the ABP module or is it a check we need to add to app/adBlock.js?

@diracdeltas diracdeltas added this to the 0.9.2dev milestone Apr 13, 2016
@bbondy
Copy link
Member

bbondy commented Apr 14, 2016

I thought we already had this check but apparently not. We should just do an extra check to make sure the details.url host is different from the firstPartyUrl.host in app/adBlock.js.

@bbondy
Copy link
Member

bbondy commented Sep 19, 2017

re-opening, I think a rule like this will fix: @@||adops.com$~third-party

@bbondy bbondy modified the milestones: 0.9.2dev, 0.21.x (Nightly Channel) Sep 19, 2017
bbondy added a commit to brave/adblock-lists that referenced this issue Sep 19, 2017
@bbondy bbondy modified the milestones: 0.21.x (Nightly Channel), 0.20.x (Developer Channel) Sep 19, 2017
bbondy added a commit to brave/adblock-lists that referenced this issue Sep 19, 2017
@bbondy
Copy link
Member

bbondy commented Sep 19, 2017

mostly fixed by the filtering rule committed above, but some things are still getting blocked that shouldn't (logo). I think it's a bug in the ad block library and could be a more generalized problem so blocking on this since the first party filtering change first lands in 0.20.x too.

@bbondy
Copy link
Member

bbondy commented Sep 19, 2017

Never mind, I just tested with combined data file and it's working. I was testing before with custom adblock rule exception which runs in an independent instance

@cezaraugusto
Copy link
Contributor

heads-up I added a basic test plan please add more checks in case mine is incomplete. Thanks

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.