mirror of
https://github.com/librespot-org/librespot.git
synced 2026-04-27 08:15:50 +03:00
[GH-ISSUE #1168] Write credentials to cache only if they are changed #539
Labels
No labels
A-Alsa
SpotifyAPI
Tokio 1.0
audio
bug
can't reproduce
compilation
dependencies
duplicate
enhancement
good first issue
help wanted
high priority
imported
imported
invalid
new api
pull-request
question
reverse engineering
wiki
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/librespot#539
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @beaverking1212 on GitHub (May 17, 2023).
Original GitHub issue: https://github.com/librespot-org/librespot/issues/1168
Is your feature request related to a problem? Please describe.
A minor problem. When login with cached credentials and
store_credentialsistruein call toSession::connect()the file with cached credentials is written every timeSession::connect()is called. The contents of the file usually do not change with these writes.Describe the solution you'd like
The file with cached credentials should be written only if cached credentials have changed (have been updated by Spotify).
Describe alternatives you've considered
Set
store_credentialstofalseand use code that writes credentials to cache if they are changed.gitlab.freedesktop.org/gstreamer/gst-plugins-rs/-@ea26d0d6d5/audio/spotify/src/common.rs (L162)Additional context
Problem was noticed in: https://gitlab.freedesktop.org/gstreamer/gst-plugins-rs/-/merge_requests/1195
@beaverking1212 commented on GitHub (May 17, 2023):
@gdesmott