Opened 5 years ago

Closed 22 months ago

#472 closed change (rejected)

Issue reporter - Asking the user to reload the page if issue is due to ABP starting late

Reported by: Crits Assignee:
Priority: P3 Milestone:
Module: Adblock-Plus-for-Firefox Keywords:
Cc: smultron45@…, trev, famlam Blocked By: #362
Blocking: Platform: Firefox
Ready: yes Confidential: no
Tester: Verified working: no
Review URL(s):

Description (last modified by mapx)

Background

Starting from ABP for Firefox 2.6, ABP start is delayed when Firefox starts, letting a few ads slip through. ABP start is planned to be sped up in the future, but it's likely it will still not have time to block a few undesirable requests.

However, many users are reporting ads through the issue reporter, not because of a filter issue, but because ABP had started late (and they are not bound to know that). Issue reports are thus quickly clogged by useless reports, and it can be pretty long to diagnose that the issue comes from #418 for these reports.
And I'm pretty sure this problem will last for long.

What to change

I had an idea on how to implement a solution:

  1. When the user report an issue, the issue reporter should take all the requests the webpage has already sent and make them go through ABP filters once again
  2. If after that the number of filtered requests is strictly superior to the number of previously blocked requests, then the issue reporter should ask the user to reload the page before trying again.

Change History (7)

comment:1 Changed 5 years ago by mapx

  • Cc smultron45@… added

comment:2 Changed 5 years ago by Crits

...small typing mistake in What to change, 1.
"should take that the webpage has already sent" => "should take all the requests the webpage has already sent"

comment:3 Changed 5 years ago by mapx

  • Description modified (diff)

comment:4 Changed 5 years ago by trev

  • Blocked By 362 added
  • Cc trev added
  • Priority changed from Unknown to P3
  • Ready set

Our content policy seems to indeed start up immediately, before the filters load. This means that Adblock Plus will see the requests happening at startup even though it doesn't block them.

#362 requires implementing some kind of "timestamp" to assign to requests (or rather a sequential counter). We can then save the "timestamp" of the last configuration change - be it filters loading or a whitelist filter added/removed. If we see that a request has a smaller timestamp than the last configuration change we can show an error page asking the user to reload page.

comment:5 Changed 5 years ago by arthur

  • Cc famlam added
  • Platform set to Unknown

comment:6 Changed 5 years ago by philll

  • Platform changed from Unknown to Firefox

comment:7 Changed 22 months ago by trev

  • Resolution set to rejected
  • Status changed from new to closed

Mass-closing all bugs in Adblock Plus for Firefox module, the codebase of Adblock Plus 3.0 belongs into Platform and User-Interface modules. Old bugs are unlikely to still apply.

Note: See TracTickets for help on using tickets.