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.

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

Passwords with # are loaded until the # character #527

Closed
1 of 6 tasks
jveiga opened this issue Mar 2, 2020 · 4 comments
Closed
1 of 6 tasks

Passwords with # are loaded until the # character #527

jveiga opened this issue Mar 2, 2020 · 4 comments
Labels
advice Advice within issues that people could find helpful later on bug A functionality or parts of a program that do not work as intended wontfix Issues that will not be fixed under any circumstances
Milestone

Comments

@jveiga
Copy link

jveiga commented Mar 2, 2020

Description
When running spotifyd with a password containing # the process fails with BadCredentials.

NB: It works when loading the password through the cli
To Reproduce
Change the password to contain a # character and run spotifyd.

Expected behavior
Spotifyd should start.

Logs

Loading config from "/XXXXX/.config/spotifyd/spotifyd.conf"
No proxy specified
Using software volume controller.
error sending packet Os { code: 65, kind: Other, message: "No route to host" }
Connecting to AP "gew1-accesspoint-b-6xhq.ap.spotify.com:443"
Caught panic with message: Authentication failed with reason: BadCredentials

Compilation flags

  • dbus_mpris
  • dbus_keyring
  • alsa_backend
  • portaudio_backend
  • pulseaudio_backend
  • rodio_backend

Versions (please complete the following information):

  • OS: Mac OS Catalina 10.15.3
  • Spotifyd: tag v0.2.24
  • cargo: 1.41.0 (626f0f40e 2019-12-03)

I changed the code to print the actual password rather than the placeholder in order to debug the BadCredentials error.

@jveiga jveiga added the bug A functionality or parts of a program that do not work as intended label Mar 2, 2020
@mainrs
Copy link
Member

mainrs commented Mar 2, 2020

Related: #411

Will get fixed in v1.0 probably as I wanted to move to toml instead of ini, which would just eliminate behaviour like this.

CLI works because the values are used as is, meaning if you pass it using quotes (--password "someSPACEin BETWEEN") spotifyd uses that password. Using the keyring feature (dbus_keyring) should work as well.

@mainrs mainrs added this to the v1.0.0 milestone Mar 2, 2020
@macpijan
Copy link

Hit on the same thing, thanks for the explaination

@mainrs mainrs added the advice Advice within issues that people could find helpful later on label May 16, 2020
@stale
Copy link

stale bot commented Jan 13, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix Issues that will not be fixed under any circumstances label Jan 13, 2021
@mainrs
Copy link
Member

mainrs commented Jan 13, 2021

Resolved, #571

@mainrs mainrs closed this as completed Jan 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
advice Advice within issues that people could find helpful later on bug A functionality or parts of a program that do not work as intended wontfix Issues that will not be fixed under any circumstances
Projects
None yet
Development

No branches or pull requests

3 participants