Opened on 04/06/2018 at 08:38:26 AM

Closed on 04/06/2018 at 11:53:10 AM

#6560 closed defect (fixed)

Dev builds of ABP for Firefox aren't being updated to the latest version

Reported by: arthur Assignee: tlucas
Priority: P2 Milestone:
Module: Automation Keywords:
Cc: sebastian, tlucas Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

https://codereview.adblockplus.org/29744579/

Description (last modified by kzar)

Environment

Mozilla Firefox 59.0.2

How to reproduce

  1. Go to https://downloads.adblockplus.org/devbuilds/adblockplusfirefox/
  2. Install Adblock Plus for Firefox 3.0.2.1997 or Adblock Plus for Firefox 3.0.2.1998
  3. Go to about:addons
  4. Click the cog wheel and then on "Check for Updates"

Observed behaviour

Firefox doesn't find any updates.

Expected behaviour

Firefox should update to the latest dev build (currently 3.0.2.2002).

Notes

Attachments (0)

Change History (6)

comment:1 Changed on 04/06/2018 at 08:49:04 AM by tlucas

I can reproduce the behavior, although this is quiet odd.

When you go to about:debugging#addons and follow Manifest URL in the extension Adblock Plus development build, you see that update_url is set to the correct URL, i.e. https://downloads.adblockplus.org/devbuilds/adblockplusfirefox/updates.json, which (currently) yields the following:

{
  "addons": {
    "devbuild@adblockplus.org": {
      "updates": [
        {
          "version": "3.0.2.2002",
          "update_link": "https://downloads.adblockplus.org/devbuilds/adblockplusfirefox/adblockplusfirefox-3.0.2.2002.xpi" },
          "applications": {
          "gecko": { "strict_min_version": "51.0" }
        }
      ]
    }
  }
}

So - the latest devbuild is listed, but apparently firefox refuses to recognize it?

comment:2 Changed on 04/06/2018 at 09:01:32 AM by tlucas

  • Component changed from Unknown to Automation
  • Owner set to tlucas
  • Priority changed from Unknown to P2
  • Ready set

Found the error, apparently the content of "updates.json" is not well formed.

comment:3 Changed on 04/06/2018 at 09:08:50 AM by tlucas

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

comment:4 Changed on 04/06/2018 at 09:51:45 AM by kzar

  • Description modified (diff)

comment:5 Changed on 04/06/2018 at 11:52:40 AM by abpbot

A commit referencing this issue has landed:
Issue 6560 - fix malformed updates.json

comment:6 Changed on 04/06/2018 at 11:53:10 AM by tlucas

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

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