release: version 0.6.7 🚀

HISTORY.md

@@ -5,10 +5,33 @@ Changelog
 (unreleased)
 ------------
 
+Fix
+~~~
+- Attempted bugfix for crashing process on invalid spotify access token,
+  refs NOISSUE. [Simon Diesenreiter]
+
+
+0.6.6 (2025-07-20)
+------------------
+
+Fix
+~~~
+- Add additional logging, refs NOISSUE. [Simon Diesenreiter]
+
+Other
+~~~~~
+
+
+0.6.5 (2025-07-20)
+------------------
+
 Fix
 ~~~
 - Configurable Cron schedules, refs NOISSUE. [Simon Diesenreiter]
 
+Other
+~~~~~
+
 
 0.6.4 (2025-07-20)
 ------------------

song_of_the_day/Program.cs

@@ -86,18 +86,21 @@ likePlaylistCheckTimer.OnOccurence += async (s, ea) =>
     {
         if (!await spotifyApiClient.IsUserAuthenticatedAsync(user))
         {
+            logger.LogWarning($"User {user.LdapUserName} is not authorized with Spotify - skipping playlist sync");
             continue;
         }
         var foundPlaylist = dci.SmartPlaylistDefinitions?.Where(p => p.CreatedBy == user).ToList().Where(p => p.IsThisUsersLikedSongsPlaylist).SingleOrDefault();
         if (foundPlaylist == null)
         {
-            var title = $"{user.PreferredName}'s liked Songs";
+            logger.LogWarning($"Creating liked songs playlist for user {user.LdapUserName}");
+            var title = $"{user.PreferredName}'s liked songs";
             var description = $"A collection of the songs liked by {user.PreferredName} on their 'Song of the day' server instance.";
             // playlist does not exist yet, creating it
             var newPlaylist = await (await spotifyApiClient.WithUserAuthorizationAsync(user)).CreateSpotifyPlaylist(title, description, false, true, user);
             await playlistSynchronizer.SynchronizePlaylistAsync(newPlaylist);
             needsSaving = true;
         }
+        logger.LogWarning($"Syncing playlists for user {user.LdapUserName}");
         await playlistSynchronizer.SynchronizeUserPlaylistsAsync(user);
     }
 
@@ -252,6 +255,7 @@ messageSyncTimer.OnOccurence += async (s, ea) =>
 
 // only start interaction timers in production builds
 // for local/development testing we want those disabled
+    likePlaylistCheckTimer.Start();
 if (!app.Environment.IsDevelopment())
 {
     logger.LogTrace("Starting timer for scheduled processes.");
@@ -259,7 +263,6 @@ if (!app.Environment.IsDevelopment())
     userIntroTimer.Start();
     pickOfTheDayTimer.Start();
     ldapAssociationTimer.Start();
-    likePlaylistCheckTimer.Start();
 }
 else
 {

song_of_the_day/SpotifyIntegration/SpotifyApiClient.cs

@@ -110,7 +110,7 @@ public class SpotifyApiClient
 
     public async Task<string> GetValidAuthorizationTokenAsync(User user)
     {
-        if (string.IsNullOrEmpty(user.SpotifyAuthAccessToken))
+        if (string.IsNullOrEmpty(user.SpotifyAuthAccessToken) || string.IsNullOrEmpty(user.SpotifyAuthRefreshToken))
         {
             // user either never connected Spotify or we failed to refresh token - user needs to re-authenticate
             return string.Empty;

song_of_the_day/VERSION

@@ -1 +1 @@
-0.6.5
+0.6.7