[PR #213] fix(domain): handle missing serverTime in Spotify ServerTimeResponse #215

New issue

Open

opened 2026-02-27 19:26:48 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 19:26:48 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Lambada10/SongSync/pull/213
Author: @Eutalix
Created: 2/12/2026
Status: 🔄 Open

Base: master ← Head: master

📝 Commits (2)

34c61cf fix(domain): handle missing serverTime in Spotify ServerTimeResponse
ef8bd31 fix(spotify): add system time fallback for TOTP generation

📊 Changes

2 files changed (+18 additions, -9 deletions)

View changed files

📝 app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/spotify/SpotifyAPI.kt (+16 -7)
📝 app/src/main/java/pl/lambada/songsync/domain/model/lyrics_providers/spotify/WebPlayerToken.kt (+2 -2)

📄 Description

Fixed the MissingFieldException crash caused by Spotify removing the serverTime field from their API.
Updated WebPlayerToken.kt to make serverTime optional (defaults to 0), preventing the serialization error.
Verified that the app can now successfully fetch the access token and lyrics again.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/Lambada10/SongSync/pull/213 **Author:** [@Eutalix](https://github.com/Eutalix) **Created:** 2/12/2026 **Status:** 🔄 Open **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (2) - [`34c61cf`](https://github.com/Lambada10/SongSync/commit/34c61cf05f7d1a12d9c482be2d3bf7f213d60566) fix(domain): handle missing serverTime in Spotify ServerTimeResponse - [`ef8bd31`](https://github.com/Lambada10/SongSync/commit/ef8bd310455b7a11d892883bd7d0ab62e5d2db39) fix(spotify): add system time fallback for TOTP generation ### 📊 Changes **2 files changed** (+18 additions, -9 deletions) <details> <summary>View changed files</summary> 📝 `app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/spotify/SpotifyAPI.kt` (+16 -7) 📝 `app/src/main/java/pl/lambada/songsync/domain/model/lyrics_providers/spotify/WebPlayerToken.kt` (+2 -2) </details> ### 📄 Description - Fixed the `MissingFieldException` crash caused by Spotify removing the `serverTime` field from their API. - Updated `WebPlayerToken.kt` to make `serverTime` optional (defaults to 0), preventing the serialization error. - Verified that the app can now successfully fetch the access token and lyrics again. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>