mirror of
https://github.com/spotipy-dev/spotipy.git
synced 2026-04-26 16:15:51 +03:00
[PR #832] [CLOSED] Fixes "Only valid bearer authentication supported, reason: None" #1034
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#1034
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/832
Author: @stephanebruckert
Created: 6/19/2022
Status: ❌ Closed
Base:
master← Head:fix-token-as-dict📝 Commits (1)
43b2c2bget_access_token() to return token instead of dict by default📊 Changes
6 files changed (+9 additions, -7 deletions)
View changed files
📝
CHANGELOG.md(+2 -0)📝
examples/app.py(+1 -1)📝
spotipy/client.py(+2 -2)📝
spotipy/oauth2.py(+2 -2)📝
spotipy/util.py(+1 -1)📝
tests/unit/test_oauth.py(+1 -1)📄 Description
Fixes https://github.com/plamere/spotipy/issues/687, https://github.com/plamere/spotipy/issues/828
To keep backward-incompatibility we shouldn't have made
as_dict=Truethe default inget_access_tokenmethods. This PR swaps it back toFalse. Therefore this must be seen as a bugfix rather than a new backward-incompatible change.Normally, this shouldn't have too much effect on new code as we now don't recommend to get the token independently via
get_access_token()in order to authenticate (see README).If you have been using
get_access_token()in new code, you should now useget_access_token(as_dict=True).🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.