Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Commit

Permalink
Adblock tweaks
Browse files Browse the repository at this point in the history
Just some small cleanup so things download from where they should.

Auditors: @diracdeltas

Test Plan: will be tested in the context of other adblock tasks.
  • Loading branch information
bbondy committed Oct 14, 2016
1 parent 45f7075 commit bc01069
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion app/adBlock.js
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ const registerAppConfigForResource = (uuid, enabled, version) => {
* @param uuid - The uuid of the adblock datafile resource
* @param forAdblock - true if main frame URLs should be blocked
*/
module.exports.updateAdblockDataFiles = (uuid, enabled, version = 1, shouldCheckMainFrame = false) => {
module.exports.updateAdblockDataFiles = (uuid, enabled, version = 2, shouldCheckMainFrame = false) => {
registerAppConfigForResource(uuid, enabled, version)
if (!adblockInstances.has(uuid)) {
const parser = new ABPFilterParser()
Expand Down
6 changes: 3 additions & 3 deletions js/constants/appConfig.js
Original file line number Diff line number Diff line change
Expand Up @@ -36,15 +36,15 @@ module.exports = {
url: getTargetAboutUrl('about:flash')
},
adblock: {
alternateDataFiles: 'https://s3.amazonaws.com/adblock-data/2/{uuid}.dat',
alternateDataFiles: 'https://s3.amazonaws.com/adblock-data/{version}/{uuid}.dat',
url: 'https://s3.amazonaws.com/adblock-data/{version}/ABPFilterParserData.dat',
version: '2',
msBetweenRechecks: 1000 * 60 * 60 * 24, // 1 day
enabled: true
},
safeBrowsing: {
url: 'https://s3.amazonaws.com/safe-browsing-data/{version}/SafeBrowsingData.dat',
version: '1',
url: 'https://s3.amazonaws.com/adblock-data/{version}/SafeBrowsingData.dat',
version: '2',
msBetweenRechecks: 1000 * 60 * 60 * 24, // 1 day
enabled: true
},
Expand Down

1 comment on commit bc01069

@diracdeltas
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

++

Please sign in to comment.