Opened on 11/29/2017 at 01:43:10 PM

Closed on 12/05/2017 at 03:59:27 PM

Last modified on 12/07/2017 at 12:48:39 PM

#6108 closed defect (fixed)

No filter list is selected after a migration failure

Reported by: diegocarloslima Assignee: diegocarloslima
Priority: P2 Milestone: Adblock-Browser-for-Android-1.3.3
Module: Adblock-Browser-for-Android Keywords:
Cc: mario, tpregueiro, jwangenheim Blocked By:
Blocking: Platform: Adblock Browser for Android
Ready: yes Confidential: no
Tester: Unknown Verified working: yes
Review URL(s):

https://codereview.adblockplus.org/29626577/

Description (last modified by diegocarloslima)

We have some users that updated ABB to 1.3.x complaining about no ads being blocked. Most of the complaints are related to the fact that no filter list is selected under Settings -> Ad blocking -> Languages. This seems to be due a migration failure (so far we're unable to consistently reproduce it), which causes all the previous filter list settings to be lost. We need to add some recovery mechanism to use the default settings when a migration failure happens.

Hint for testers

Since we can't reproduce this issue (and thus can't yet fix the cause of the problem), we've implemented a mechanism to leverage configuration loss after the upgrade. As soon as ABB starts it checks for migration errors during the upgrade process. If it detects migration issues, ad blocking configuration will be reset. Thus, as soon as migration issues arise, ABB will automatically subscribe to EasyList + EasyList flavor again.

Since this is a really hard to reproduce issue, a special apk was built in order to always cause a failure when loading the filter list from the disk (after the first run, no filter is selected in Settings -> Ad blocking -> Languages) . This way, we can better verify that the 1.3.3 release candidate will recover from this kind of file corruption/migration issue

Attachments (0)

Change History (7)

comment:1 Changed on 12/01/2017 at 08:43:07 AM by mario

  • Milestone set to Adblock-Browser-for-Android-1.3.3

comment:2 Changed on 12/01/2017 at 08:26:53 PM by diegocarloslima

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:3 Changed on 12/05/2017 at 09:01:29 AM by mario

  • Description modified (diff)

comment:4 Changed on 12/05/2017 at 03:58:53 PM by abpbot

A commit referencing this issue has landed:
Issue 6108 - No filter list is selected after a migration failure

comment:5 Changed on 12/05/2017 at 03:59:27 PM by diegocarloslima

  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:6 Changed on 12/06/2017 at 01:06:43 PM by diegocarloslima

  • Description modified (diff)

comment:7 Changed on 12/07/2017 at 12:48:39 PM by traynard

  • Verified working set

LGTM

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 diegocarloslima.
 
Note: See TracTickets for help on using tickets.