mirror of
https://github.com/AnanthVivekanand/spotify-adblock.git
synced 2026-04-26 21:15:53 +03:00
[PR #78] Allow unknown domains to fix Podcast functionality #208
Labels
No labels
cannot reproduce
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/spotify-adblock#208
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/AnanthVivekanand/spotify-adblock/pull/78
Author: @imtherainman
Created: 2/9/2023
Status: 🔄 Open
Base:
master← Head:master📝 Commits (1)
f960fbaAllow unknown domains (#1)📊 Changes
2 files changed (+9 additions, -3 deletions)
View changed files
📝
mitm-utils/blacklist-mitm.js(+6 -1)📝
mitm.js(+3 -2)📄 Description
I've noticed the current implementation breaks functionality for podcast audio. Looking into the logs, it appears to be due to some podcasts retrieving audio data directly from the publisher's domains.
For example, NPR podcasts send requests to

edgeX.pod.npr.org, and New York Times usesnyt.simplecastaudio.com:Since these domains do not appear in either the whitelist or blacklist, they are blocked by
mitm.js.Seeing as there's no way to possibly whitelist every one of these publisher domains, the only workaround I can think of is to allow unknown domains by default, and update
blacklist-mitm.jsaccordingly whenever an advertisement request slips through. I've added the following domains to the blacklist based on what was previously blocked as an unknown domain that would otherwise now be allowed:spclient.wg.spotify.com/gabo-receiver-service/**(Spotify user data tracking)spclient.wg.spotify.com/sponsoredplaylist/**(blocks the "sponsered by" visual ads in playlists)**/*sentry.io*/**(user data tracking)**/*googleapis*/**(user data tracking)**/*doubleclick*/**(ads)I've been using this updated version for a few weeks now without issues, but welcome any feedback or alternative approach. Thanks so much for maintaining this project!
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.