Opened on 03/04/2019 at 09:33:44 AM
Closed on 08/29/2019 at 05:43:18 PM
#7322 closed change (rejected)
Allow more comprehensive $document blocking
Reported by: | fanboy | Assignee: | |
---|---|---|---|
Priority: | Unknown | Milestone: | |
Module: | Platform | Keywords: | closed-in-favor-of-gitlab |
Cc: | arthur, kzar, mjethani, mapx, sebastian, greiner | Blocked By: | |
Blocking: | Platform: | Unknown / Cross platform | |
Ready: | no | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
Description (last modified by mjethani)
Environment
Adblock Plus 3.5
Chrome 71
macOS 10.12
Background
$document should block requests for the root document.
How to reproduce
- Add @@ite$image,domain=newser.com and ||yvsystem.com^$document
- Open the page http://www.newser.com/story/272065/powerful-tornado-kills-more-than-20-in-alabama.html
- Click on the Yavli Ad on the right.
Observed behaviour
Clicking the link in the ad takes you to a URL starting with http://a.yvsystem.com/ in the top frame, but ABP allows the site to load the content.
Expected behaviour
Stop the sites with $document from loading. The filter ||yvsystem.com^$document should block the request to the URL starting with http://a.yvsystem.com/.
This will stop sites using the redirection through adservers method, just another tool to stop this.
Notes
This feature was implemented in uBlock Origin in response to a report about malware/phishing sites using top-level redirection (uBlock issue 1013).
Attachments (0)
Change History (12)
comment:2 Changed on 03/04/2019 at 01:20:23 PM by mapx
- Cc arthur kzar mjethani mapx added
comment:3 Changed on 03/06/2019 at 12:35:44 PM by mjethani
comment:4 Changed on 03/06/2019 at 11:22:50 PM by fanboy
Yeah, essentially I'm wanting $document in ABP to block sites outright (similar to ubo), given some adserver sites will use them to redirect.
comment:5 Changed on 03/23/2019 at 11:04:49 PM by SMed79
uBlock wiki > Strict blocking ~ https://github.com/gorhill/uBlock/wiki/Strict-blocking
comment:6 Changed on 03/25/2019 at 12:16:17 PM by mjethani
- Cc sebastian added
comment:7 Changed on 03/25/2019 at 12:17:41 PM by mjethani
- Component changed from Unknown to Platform
comment:9 Changed on 03/25/2019 at 12:37:27 PM by mjethani
@fanboy I have rewritten the issue description a bit to conform to how we write these generally, please make corrections as you see fit.
comment:10 Changed on 03/26/2019 at 12:03:56 AM by sebastian
- Cc greiner added
- Type changed from defect to change
This feature has no use for ad blocking but could enable filter lists that protect the user from accidentally accessing malicious websites. In the past, our stance here was that this isn't the purpose of Adblock Plus, and there are other extensions for this kind of blocking.
I personally don't have a strong opinion at the current point, but we should probably discuss this feature request some more before moving forward. In any case it seems low priority to me.
comment:11 Changed on 03/26/2019 at 12:56:20 PM by greiner
Unfortunately, I can't reproduce it but from what I understand, you're saying that the popup is not being blocked and that you'd like to work around that by instead blocking the entire page that's being opened.
If so, wouldn't it make more sense to add an option that closes the tab or prevents it from being opened, similar to popup blocking works, instead of blocking the entire page in it?
In general, I think we should be very careful when retroactively changing the behavior of $document and instead look for alternatives. Also, I'm not too convinced about uBlock Origin's approach because it seems very obtrusive.
comment:12 Changed on 08/29/2019 at 05:43:18 PM by sebastian
- Keywords closed-in-favor-of-gitlab added
- Resolution set to rejected
- Status changed from new to closed
Sorry, but we switched to GitLab. If this issue is still relevant, please file it again in the new issue tracker.
To clarify what @fanboy means:
(Correct me if I'm wrong @fanboy)
In ABP we explicitly ignore navigations in the top frame.