search

Polochon-street / blissify-rs

bliss bindings for MPD.
30 stars 6 forks source link

Existence of password support is unclear #37

Closed animaldaydream closed 1 year ago

animaldaydream commented 1 year ago

There is no documentation for the JSON config (or I can't find it anyway). I assumed I could set up a password there.

Afterwards I tried to set the password on the MPD_HOST environment variable, but I am getting this error:

[2023-03-26T18:18:44Z WARN  blissify] Could not find any MPD_PORT environment variable set. Defaulting to 6600.
Error: failed to lookup address information: Name or service not known

When the password is not set, I get this error instead:

[2023-03-26T18:18:57Z WARN  blissify] Could not find any MPD_HOST environment variable set. Defaulting to 127.0.0.1.
[2023-03-26T18:18:57Z WARN  blissify] Could not find any MPD_PORT environment variable set. Defaulting to 6600.
Error: permission error (`you don't have permission for "random"') at 0

The only option I have is to not set a password. [Even though I encrypt the connection through Wireguard, I have a VPN connection set up through a VPS as well, so this is very inconvenient regardless.]

PD: Apparently Unix sockets through the envorinment variable aren't supported either (outputs first error), but I'll leave that for the next person to make an issue.

Polochon-street commented 1 year ago

I've changed that here https://github.com/Polochon-street/blissify-rs/pull/38, could you try and checkout that branch (support-standard-mpd-host) to see if it works for you? I've changed things around so the connection mechanism follows https://mpd.readthedocs.io/en/latest/client.html#connecting-to-mpd more closely, and password should be supported now :)

animaldaydream commented 1 year ago

I can confirm it works! It also works with a socket now. No errors in the terminal. Thank you!

Polochon-street commented 1 year ago

Awesome. Just released a new version, so people can use it without checking out that branch. Thanks for the feedback!