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

  1. Add @@ite$image,domain=newser.com and ||yvsystem.com^$document
  2. Open the page http://www.newser.com/story/272065/powerful-tornado-kills-more-than-20-in-alabama.html
  3. 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:1 Changed on 03/04/2019 at 01:19:00 PM by mapx

  • Description modified (diff)

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

To clarify what @fanboy means:

  1. There is a filter in EasyList ||yvsystem.com^$document
  2. Clicking the link in the ad takes you to a URL starting with http://a.yvsystem.com/ in the top frame
  3. uBlock Origin supposedly blocks the request so the navigation is prevented

(Correct me if I'm wrong @fanboy)

In ABP we explicitly ignore navigations in the top frame.

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:8 Changed on 03/25/2019 at 12:36:30 PM by mjethani

  • Description modified (diff)

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.

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from (none).
 
Note: See TracTickets for help on using tickets.