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

Check password save #945

Open
jbonnet opened this issue Nov 22, 2017 · 7 comments
Open

Check password save #945

jbonnet opened this issue Nov 22, 2017 · 7 comments
Labels

Comments

@jbonnet
Copy link
Member

jbonnet commented Nov 22, 2017

It is not working when PUT http://sp.int3.sonata-nfv.eu:32001/api/v2/users?username=<username> is used

@jbonnet jbonnet added the bug label Nov 22, 2017
@dang03
Copy link
Collaborator

dang03 commented Nov 22, 2017

Hi @jbonnet
If you are trying to update the user's password, the PUT method to /api/v2/users? won't do it. It was only intended to update users private and public key. However if this functionality is needed, I can implement it (it requires adding a new case in the code).

@jbonnet
Copy link
Member Author

jbonnet commented Nov 22, 2017

Ha! Right, @dang03, we need that...
Thanks,

@dang03
Copy link
Collaborator

dang03 commented Nov 22, 2017

Ok, there I go!

@dang03
Copy link
Collaborator

dang03 commented Nov 22, 2017

@jbonnet How is the body of the message you send to the endpoint?
Is the full user registration message (including a new password) or it only has the new password key: value?
Thanks

@jbonnet
Copy link
Member Author

jbonnet commented Nov 22, 2017

@dang03 the current API is to update all fields...

dang03 added a commit that referenced this issue Nov 24, 2017
Updated UM User update endpoint (issue #945)
@dang03
Copy link
Collaborator

dang03 commented Nov 24, 2017

Solved.
The 'username' and its 'id' number can't be modified/updated.

@jbonnet
Copy link
Member Author

jbonnet commented Nov 24, 2017

+1, @dang03
@pkarkazis @mzacharioudakis can you please retest this?

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

No branches or pull requests

2 participants