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

Commit

Permalink
Bump to HTTPS Everywhere 5.1.9
Browse files Browse the repository at this point in the history 
Fix #1692

CC @garvankeeley @SergeyZhukovsky
  • Loading branch information
@diracdeltas
diracdeltas committed May 23, 2016
1 parent 981d1b0 commit c1b042b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion js/constants/appConfig.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ module.exports = {
},
httpsEverywhere: {
url: 'https://s3.amazonaws.com/https-everywhere-data/{version}/httpse.json',
version: '5.1.6', // latest stable release from https://eff.org/https-everywhere
version: '5.1.9', // latest stable release from https://eff.org/https-everywhere
msBetweenRechecks: 1000 * 60 * 60 * 24, // 1 day
enabled: true
},
Expand Down

4 comments on commit c1b042b

@bbondy
Copy link
Member

@bbondy bbondy commented on c1b042b May 23, 2016

Choose a reason for hiding this comment

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

++

@bbondy
Copy link
Member

@bbondy bbondy commented on c1b042b May 23, 2016

Choose a reason for hiding this comment

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

This version only needs to be changed for file format changes I think by the way. You could just replace the old location and it would get the new one by etag comparison. I'm fine either way though.

@diracdeltas
Copy link
Member Author

Choose a reason for hiding this comment

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

yeah, the only reason to bump the version right now is so that we can track which version of HTTPS Everywhere upstream this corresponds to. but that hasn't been necessary so far, so i'm fine with just updating the same file path until there is a breaking change.

@bbondy
Copy link
Member

@bbondy bbondy commented on c1b042b May 24, 2016

Choose a reason for hiding this comment

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

either way, you can continue as you are.

Please sign in to comment.