Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix embedded cover art #2

Merged
merged 3 commits into from
Mar 16, 2021
Merged

Fix embedded cover art #2

merged 3 commits into from
Mar 16, 2021

Conversation

iRet
Copy link

@iRet iRet commented Mar 12, 2021

The PR is to fix embedded cover art
Inspired by hbashton/spotify-ripper#53

@scaronni scaronni merged commit 121c970 into scaronni:master Mar 16, 2021
@scaronni
Copy link
Owner

Thanks for the merge request. I've not been using spotify-ripper for a few weeks and now I'm having issues with spotipy, so I can't really test and create a new tag:

Traceback (most recent call last):
  File "/usr/lib64/python3.9/threading.py", line 954, in _bootstrap_inner
    self.run()
  File "/usr/lib/python3.9/site-packages/spotify_ripper/ripper.py", line 163, in run
    init_spotipy(self.session.user.canonical_name)
  File "/usr/lib/python3.9/site-packages/spotify_ripper/spotipy_integration.py", line 17, in init_spotipy
    token = spotipy.util.prompt_for_user_token(username, scope)
  File "/usr/lib/python3.9/site-packages/spotipy/util.py", line 96, in prompt_for_user_token
    token_info = sp_oauth.validate_token(sp_oauth.cache_handler.get_cached_token())
  File "/usr/lib/python3.9/site-packages/spotipy/oauth2.py", line 325, in validate_token
    token_info = self.refresh_access_token(
  File "/usr/lib/python3.9/site-packages/spotipy/oauth2.py", line 554, in refresh_access_token
    raise SpotifyException(response.status_code,
spotipy.exceptions.SpotifyException: http status: 400, code:-1 - Couldn't refresh token: code:400 reason:Bad Request, reason: {'Authorization': 'Basic YjJhNmY1NDkzMzlkNDhjMmJlM2MwYzUwMWZhMmM4MzA6NTE0YjM0OWNhZTFjNDBkNmI1MmM0OThkNWZiNjFhZmE='}

Not clear to me what has changed. Any hint?

@scaronni
Copy link
Owner

Ok fixed, I just had to delete the cache file.

@iRet
Copy link
Author

iRet commented Mar 18, 2021

@scaronni thanks for sharing!

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.

2 participants