search

mopidy / mopidy-spotify

Mopidy extension for playing music from Spotify
https://mopidy.com/ext/spotify/
Apache License 2.0
933 stars 109 forks source link

Use the long-form Spotify URL for user playlists. #299

Closed kingosticks closed 3 years ago

kingosticks commented 3 years ago

For logged in user 'foo', Web API endpoint "me/playlists" returns the same as "users/foo/playlists" (including headers). When we use the former, the "next" links in the response always use the latter. This leaves us with a mix of URL formats in our cache and will make identifying/invalidating these entries harder.

This change ensures all the URLs look the same. No behaviour has changed.

This will make cache invalidation in #287 a little simpler and makes sense to do anyway.

codecov[bot] commented 3 years ago

Codecov Report

Merging #299 (5119e91) into master (f4317b1) will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #299   +/-   ##
=======================================
  Coverage   96.68%   96.68%           
=======================================
  Files          13       13           
  Lines        1207     1207           
=======================================
  Hits         1167     1167           
  Misses         40       40
Impacted Files Coverage Δ
mopidy_spotify/web.py 96.08% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update f4317b1...5119e91. Read the comment docs.