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

Add discord's media domain to blocklist-discord.txt using the ABP blocklist style. #4

Merged
merged 1 commit into from
Feb 9, 2024

Conversation

MarcSollie
Copy link
Contributor

Discord has migrated to using around 9999 per server location of subdomains of the .discord.media domain to facilitate connecting to voice calls and voice channels, even when the main application is not accessible. This means that 9999 * 14 locations = ~14000 subdomains have to be blocked, which has to be kept up for every location update.

Since March of 2023 Pihole supports limited ABP style blocklists. (Pull request: pi-hole/pi-hole/pull/5179)

This commit makes use of this feature to block all discord.com (sub)domains and all discord.media subdomains, ensuring the application is completely and properly blocked, even when new Voice Server locations are added, or when the app was opened once on another network.

This will not be backwards compatible with earlier versions of PiHole. If maintaining backwards compatibility with 1yrd old versions is necessary, I am able to revert the blocklist-discord.txt file and make this change as a seperate, new file.

Discord has migrated to using around 9999 per server location of subdomains of the .discord.media domain to facilitate connecting to voice calls and voice channels, even when the main application is not accessible.

Since 2023 Pihole supports limited ABP style blocklists. This commit makes use of this feature to block all discord.com domains and all discord.media subdomains, ensuring the application is completely and properly blocked, even when the app was opened once on another network.
@nickoppen nickoppen merged commit 03e0b51 into nickoppen:master Feb 9, 2024
@nickoppen
Copy link
Owner

Thankyou Marc.

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.

2 participants