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.

Sign up for GitHub

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

Jump to bottom

Improve credentials cache #357

Merged
merged 5 commits into from
Apr 21, 2023
Merged

Improve credentials cache #357

merged 5 commits into from
Apr 21, 2023

Conversation

beaverking1212
Copy link
Contributor

Always cache credentials and use dedicated folder for credentials cache. Folder for credentials cache has mode 700.
For #353

@beaverking1212 beaverking1212 mentioned this pull request Apr 21, 2023
Open
kingosticks
kingosticks requested changes Apr 21, 2023
View reviewed changes
mopidy_spotify/backend.py Outdated Show resolved Hide resolved
@kingosticks kingosticks added the A-spotifyaudiosrc Area: spotifyaudiosrc (GStreamer/librespot) label Apr 21, 2023
@beaverking1212
Copy link
Contributor Author

Fixed

Extension().get_data_dir(self.backend._config) makes the folder it returns. Now folders are created on Mopidy start.

@kingosticks kingosticks merged commit 984151a into mopidy:master Apr 21, 2023
@kingosticks
Copy link
Member

Brilliant, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kingosticks kingosticks kingosticks requested changes

Assignees
No one assigned
Labels
A-spotifyaudiosrc Area: spotifyaudiosrc (GStreamer/librespot)
Projects
None yet
Milestone
5.0 (Replace libspotify)
Development

Successfully merging this pull request may close these issues.
2 participants
@beaverking1212 @kingosticks