mirror of
https://github.com/spotipy-dev/spotipy.git
synced 2026-04-26 16:15:51 +03:00
[GH-ISSUE #1071] _regex_spotify_url needs update because of region specific path #633
Labels
No labels
api-bug
bug
dependencies
documentation
duplicate
enhancement
external-ide
headless-mode
implicit-grant-flow
invalid
missing-endpoint
pr-welcome
private-api
pull-request
question
spotipy3
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/spotipy#633
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?
Originally created by @Commandcracker on GitHub (Jan 14, 2024).
Original GitHub issue: https://github.com/spotipy-dev/spotipy/issues/1071
Originally assigned to: @dieser-niko on GitHub.
github.com/spotipy-dev/spotipy@a14a28e10c/spotipy/client.py (L121C1-L121C173)The URL has some additions in some regions (for me Germany)
https://open.spotify.com/intl-de/track/2Y0iGXY6m6immVb2ktbseM?si=22cad793e8e248fehttps://open.spotify.com/intl-de/artist/0SwO7SWeDHJijQ3XNS7xEE?si=17xfyfNGROyr18by1fh9swThe new
intl-dewill make the regx expression not match anything.Im not a regx pro but this seames to work:
⬇️ (added
.*afteropen.spotify.com\/)@dieser-niko commented on GitHub (May 3, 2024):
I think it would be better to add something more distinctive:
Added
(intl-\w\w\/)?instead of.*:The intl-<countrycode> is still just optional, so this should work with the old format as well. I'll create a pull request sooner or later to implement this.