Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove deprecated Disconnect lists from "defaults" #3451

Merged
merged 1 commit into from
May 27, 2020

Conversation

PromoFaux
Copy link
Member

By submitting this pull request, I confirm the following:

  • I have read and understood the contributors guide, as well as this entire template.
  • I have made only one major change in my proposed changes.
  • I have commented my proposed changes within the code.
  • I have tested my proposed changes, and have included unit tests where possible.
  • I am willing to help maintain this change if there are issues with it later.
  • I give this submission freely and claim no ownership.
  • It is compatible with the EUPL 1.2 license
  • I have squashed any insignificant commits. (git rebase)

What does this PR aim to accomplish?:

See #3450 for details

Removing these lists only removes 337 unique domains from a fresh install with all the lists selected. No brainer really, especially as they're deprecated!

With:

[i] Number of gravity domains: 87259 (84818 unique domains)

Without:

[i] Number of gravity domains: 84524 (84481 unique domains)

@PromoFaux PromoFaux requested a review from a team May 26, 2020 20:14
Signed-off-by: Adam Warner <me@adamwarner.co.uk>
@PromoFaux PromoFaux force-pushed the its-getting-awful-quiet-in-here branch from efe96d5 to 4f7bb7b Compare May 26, 2020 20:23
@@ -1240,8 +1238,6 @@ appendToListsFile() {
case $1 in
StevenBlack ) echo "https://raw.githubusercontent.com/StevenBlack/hosts/master/hosts" >> "${adlistFile}";;
MalwareDom ) echo "https://mirror1.malwaredomains.com/files/justdomains" >> "${adlistFile}";;
DisconTrack ) echo "https://s3.amazonaws.com/lists.disconnect.me/simple_tracking.txt" >> "${adlistFile}";;
Copy link
Contributor

Choose a reason for hiding this comment

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

Are both lists deprecated? Just wondering :)

Copy link
Member Author

Choose a reason for hiding this comment

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

Per the response from disconnect.me to the op of #3450

The list you referenced was used in our legacy products. It is not maintained, has not been updated, and is not actively distributed us. Airbrake is not included on our main tracker list (linked below) which is used by partners such as Firefox.

https://github.com/disconnectme/disconnect-tracking-protection/blob/master/services.json

I'm making the reasonable assumption that both lists are no longer updated/distributed. Besides which, when removed only 337 domains are "lost" from an install that has all lists selected.

@PromoFaux PromoFaux merged commit e43450f into development May 27, 2020
@PromoFaux PromoFaux deleted the its-getting-awful-quiet-in-here branch May 27, 2020 07:46
@pralor-bot
Copy link

This pull request has been mentioned on Pi-hole Userspace. There might be relevant details there:

https://discourse.pi-hole.net/t/initial-deployment-docker-errors/34811/15

@PromoFaux PromoFaux mentioned this pull request Jul 5, 2020
@pralor-bot
Copy link

This pull request has been mentioned on Pi-hole Userspace. There might be relevant details there:

https://discourse.pi-hole.net/t/pi-hole-5-1-released/35577/1

@pralor-bot
Copy link

This pull request has been mentioned on Pi-hole Userspace. There might be relevant details there:

https://discourse.pi-hole.net/t/reinstall-issue/43070/4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants