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

Password sent in clear text #218

Open
F00ker opened this issue Jul 24, 2015 · 3 comments
Open

Password sent in clear text #218

F00ker opened this issue Jul 24, 2015 · 3 comments

Comments

@F00ker
Copy link

F00ker commented Jul 24, 2015

On every REST call the username and password are sent in clear text, meaning the demo servers access logs are filled with login combos.

The API supports password encryption (using enc:)

And any shared jamstash.

@Hyzual
Copy link
Contributor

Hyzual commented Jul 26, 2015

Hi,

I'm not sure what you're suggesting, we do use 'enc:' when sending passwords.
Anyway, we don't have many options since Subsonic requests username and passwords for every request. Subsonic also supports HTTP Basic Authentication, I haven't tried implementing in in Jamstash yet, do you think it could solve this problem ?

@F00ker
Copy link
Author

F00ker commented Jul 26, 2015

Hi,

My access logs show a clear text password:
GET /rest/getMusicFolders.view?c=Jamstash&callback=angular.callbacks._t&f=jsonp&p=cleartext&u=cleartext&v=1.6.0

I'm not sure if basic HTTP auth would change all that much on the encryption point of view but it should slightly lighten the server load.

@Hyzual
Copy link
Contributor

Hyzual commented Jul 26, 2015

To be precise we encrypt the password when saving it, in the settings. Then we send what we saved as-is. Maybe try entering your password and saving again ?

Yes, I didn't think basic auth would improve password encryption either. I'm putting it aside for now as there are other things that need improvements, in the meantime feel free to contribute a pull request :).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants