mirror of
https://github.com/spotipy-dev/spotipy.git
synced 2026-04-26 16:15:51 +03:00
[PR #244] [MERGED] Use client credentials flow to fix broken non-authorized requests in test files #831
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#831
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/spotipy-dev/spotipy/pull/244
Author: @alisha
Created: 1/17/2018
Status: ✅ Merged
Merged: 1/16/2020
Merged by: @stephanebruckert
Base:
master← Head:master📝 Commits (4)
63eebfaUse client credentials flow to fix broken non-authorized requests9b8c90dMerge branch 'master' into master83a1cb5Fix doc and readme2ab985aLint📊 Changes
25 files changed (+111 additions, -418 deletions)
View changed files
📝
CHANGELOG.md(+4 -0)📝
README.md(+13 -6)📝
docs/index.rst(+22 -35)📝
examples/artist_albums.py(+2 -1)📝
examples/artist_discography.py(+3 -1)📝
examples/client_credentials_flow.py(+2 -2)➖
examples/deadmau5.gv(+0 -31)➖
examples/deadmau5.png(+0 -0)➖
examples/long_track_test.dat(+0 -283)📝
examples/search.py(+2 -1)📝
examples/show_album.py(+4 -6)📝
examples/show_artist.py(+4 -6)📝
examples/show_artist_top_tracks.py(+2 -3)📝
examples/show_related.py(+1 -2)📝
examples/show_track_info.py(+5 -5)📝
examples/show_tracks.py(+3 -1)📝
examples/show_user.py(+3 -1)📝
examples/simple0.py(+1 -1)📝
examples/simple1.py(+1 -1)📝
examples/simple2.py(+1 -3)...and 5 more files
📄 Description
Updated the files in the
examplesfolder to use client credentials flow instead of non-authorized requests (which, as mentioned in issue #194, is no longer supported by Spotify). Also updates this intests/tests.pyso that all tests pass.Also fixed the
test_search_timeoutfunction intests/tests.py, which was looking for arequests.ReadTimeoutexception. According to the requests docs, this should actually berequests.exceptions.Timeout.🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.