flathub / com.spotify.Client

https://flathub.org/apps/details/com.spotify.Client
74 stars 34 forks source link

Proxy settings are not saved #280

Closed amirkazemzade closed 5 months ago

amirkazemzade commented 5 months ago

Description

After changing proxy settings, there is only a Restart App button for saving it. After clicking it, the app will close and after relaunching (manually, does not run again automatically), the proxy settings are set back to default.

Before clicking Restart App

image

After relaunching manually

image

guihkx commented 5 months ago

How did you make that modal dialog appear?

Because here, proxy settings are available by clicking on your photo (on the top right corner), clicking on Settings, and then scrolling the page all the way down.

Additionally, these settings in specific are automatically saved to ~/.var/app/com.spotify.Client/config/spotify/prefs, as soon as you change add/remove any character from your proxy settings.

amirkazemzade commented 5 months ago

@guihkx

The modal dialog is the kind of view you would get when you are not logged in yet, and the only setting available before logging in is the proxy settings.

image

By the way, there is no spotify folder in my ~/.var/app/com.spotify.Client/config/ directory!

image

guihkx commented 5 months ago

Thanks for that info, I can reproduce the problem now.

This looks like a genuine bug in the app to me.

I found a workaround for it, though: Just manually create the spotify folder at any time, and then the app will be able to create the prefs file inside of it, i.e.:

mkdir -p ~/.var/app/com.spotify.Client/config/spotify
amirkazemzade commented 5 months ago

The workaround worked nicely. Thanks.

I'll put some info here that might help you solve the bug: OS: Fedora Workstation Version: 39 Install Method: Fedora Software Store

guihkx commented 5 months ago

I'm glad that worked for you, but allow me to clarify a misunderstanding: This Flatpak package is not officially endorsed (or supported) by Spotify.

We're all just community volunteers. :)

In any case, they seem to have an official channel where you can report bugs, so you could do that, if you want.

However, I think that adding a workaround here for this would be really trivial. I'll send a pull request for review.

amirkazemzade commented 5 months ago

@guihkx I knew it was not official, but I thought this bug was related to the Flatpak version. Just asking, is this a port of the Debian version that is officially supported?

guihkx commented 5 months ago

No, this is a repackage of the official Snap package.