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

If SpotifyAPI fails, do not use LastFM, end the session #483

Open
welles opened this issue Feb 25, 2023 · 1 comment
Open

If SpotifyAPI fails, do not use LastFM, end the session #483

welles opened this issue Feb 25, 2023 · 1 comment
Labels
api 📡 Related to Spotify or LastFM API task ⚒ An improvement or general task to do

Comments

@welles
Copy link

welles commented Feb 25, 2023

I've been running some overnight unattended recording sessions and several times during the sessions Spytify has asked me to re-authorize the app with Spotify API by opening the browser window.
Because I was not at the machine the timeout for the authorization of course ran out.

I don't know if this loss of authorization is a problem on Spotify's side, but what was definitely unfortunate was that Spytify just silently switched to the LastFM ID3 API and continued recording without indicating in the log or anywhere else in the UI that this had occured, which meant I could not easily see when the app had switched the API or which songs had been tagged by LastFM since then.

I would love it if there was an obvious error message when the authorization failed and an option in the settings to stop spying (and maybe Spotify playback?) when it happens.

@welles
Copy link
Author

welles commented Feb 25, 2023

Correction: Apparently there is a popup that tells you when the authorization fails. No idea how I missed that! Still, I would prefer it if Spytify would stop spying when it happens instead of just continuing with LastFM.

@jwallet jwallet added task ⚒ An improvement or general task to do api 📡 Related to Spotify or LastFM API and removed bug 🐞 labels Apr 4, 2023
@jwallet jwallet changed the title Spytify loses Spotify API authorization and switches to LastFM without notifying If SpotifyAPI fails, do not use LastFM, end the session Apr 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api 📡 Related to Spotify or LastFM API task ⚒ An improvement or general task to do
Projects
None yet
Development

No branches or pull requests

2 participants