Do not reset authentication if last.fm returns auth failure (#27)

It is currently unclear whether last.fm's error code 4 (no permissions) is only an intermittent error or not.
gix · Mar 28, 2021 · 69220be · 69220be
1 parent 8ae9617
commit 69220be
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,12 @@
 # Changelog
 All notable changes to this project will be documented in this file.
 
+## [1.5.0] - 2021-03-28
+### Changed
+- A permission denied error from last.fm only temporarily suspends scrobbling
+  instead of resetting authorization. This may help with intermittent errors
+  on last.fm's side. (#27)
+
 ## [1.4.0] - 2020-11-22
 ### Fixed
 - If authorization with last.fm is rejected, the unauthorized state is now

diff --git a/src/foo_scrobble/LastfmScrobbleService.cpp b/src/foo_scrobble/LastfmScrobbleService.cpp
@@ -405,6 +405,9 @@ void LastfmScrobbleService::HandleResponseStatus(lastfm::Status status)
             break;
 
         case lastfm::Status::AuthenticationFailed:
+            PauseProcessing(minutes(1));
+            break;
+
         case lastfm::Status::InvalidSessionKey:
             state_ = State::UnauthenticatedIdle;
             ClearSessionKeyLocked();