Opened on 01/20/2015 at 01:44:42 PM

Closed on 01/21/2015 at 10:50:57 AM

Last modified on 01/21/2015 at 12:25:18 PM

#1834 closed change (worksforme)

Add public domain suffix import to buildtools

Reported by: kzar Assignee:
Priority: P3 Milestone:
Module: Automation Keywords:
Cc: sebastian, trev Blocked By:
Blocking: Platform: Unknown
Ready: yes Confidential: no
Tester: Verified working:
Review URL(s):

Description (last modified by sebastian)

Background

Mozilla maintain a list of public domain suffixes online. We use this in our extensions, but we store it in a different format.

Currently keeping our copies of the list up to date is a manual process and this is time consuming. Hence our list is outdated most of the time.

What to change

Implement .build.py -t chrome getpublicsuffixlist. This command should:

  1. Download Mozilla's Public Suffix List
  2. Convert it to the format used internally by Adblock Plus
  3. Update lib/publicSuffixList.js

Attachments (0)

Change History (7)

comment:1 Changed on 01/20/2015 at 01:46:32 PM by kzar

  • Cc kzar added

comment:2 Changed on 01/20/2015 at 01:59:21 PM by kzar

  • Cc kzar removed

comment:3 Changed on 01/20/2015 at 02:00:11 PM by kzar

  • Description modified (diff)

comment:4 Changed on 01/20/2015 at 02:06:01 PM by sebastian

  • Description modified (diff)
  • Priority changed from Unknown to P3
  • Ready set
  • Summary changed from Add public domain suffix import feature to buildtools to Add public domain suffix import to buildtools

comment:5 Changed on 01/21/2015 at 10:50:57 AM by trev

  • Cc trev added
  • Resolution set to worksforme
  • Status changed from new to closed

It's build.py -t chrome updatepsl actually.

comment:6 Changed on 01/21/2015 at 11:13:20 AM by sebastian

Interesting, didn't know that. Do you think we should update the public suffix list during the release process, just like we do for translations?

comment:7 Changed on 01/21/2015 at 12:25:18 PM by trev

Yes, that probably makes sense.

Note that there is a second copy of it in the libadblockplus repository, we probably want to keep it in sync.

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.