Skip to content

Conversation

RayyanSeliya
Copy link
Contributor

Problem

While setting up the ListenBrainz project and integrating Spotify, I encountered an issue with the redirect url http://localhost:8100/settings/music-services/spotify/callback/. Spotify detects this url as not secure in the Developer Dashboard, because they have updated their way of accepting the callback url as I have attached the screenshot .
Screenshot 2025-05-01 232324

Refer the screenshot below for the updated guidlines of spotify :
Screenshot 2025-05-01 232254

I also noticed this url is hardcoded in config.py.sample , so I believe both the docs and this config need an update which will help newcomers for local development

Solution

successfully updated server development docs by updating the callback/redirect url of spotify to : https://127.0.0.1:8100/settings/music-services/spotify/callback/ as suggested by @amCap1712 to use 127.0.0.1 instead of localhost
and also updated the config.py.sample :
image

and please ignore this commit : c632f07903b28312f2c407d0d59b5bad517f1d3e

@MonkeyDo
Copy link
Member

For posterity, we are waiting for Spotify to fix their buggy URL validation.
See https://community.spotify.com/t5/Spotify-for-Developers/Local-Redirect-URI-does-not-accept-hyphen-minus/m-p/6929053 for shouting into the void

@RayyanSeliya
Copy link
Contributor Author

For posterity, we are waiting for Spotify to fix their buggy URL validation.
See https://community.spotify.com/t5/Spotify-for-Developers/Local-Redirect-URI-does-not-accept-hyphen-minus/m-p/6929053 for shouting into the void

Yeah @MonkeyDo we have to wait I think I got this error while setting up on local , getting nostalgia how I started contributing to metabrainz 🥹! Thx

Copy link
Member

@MonkeyDo MonkeyDo left a comment

Choose a reason for hiding this comment

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

Spotify has fixed their URI validation on their end, so this is now ready after a small change.

@MonkeyDo MonkeyDo requested a review from amCap1712 August 12, 2025 15:39
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.

3 participants