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.

Sign up for GitHub

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

Jump to bottom

Change password not working #370

Closed
mmeloni opened this issue Jul 3, 2020 · 1 comment · Fixed by #438
Closed

Change password not working #370

mmeloni opened this issue Jul 3, 2020 · 1 comment · Fixed by #438
Labels
bug Something isn't working Component::immudb Server component of immudb Feature::multidb Multi database support
Milestone
v0.7

Comments

@mmeloni
Copy link
Contributor

mmeloni commented Jul 3, 2020 •
edited
Loading

What happened
Change password return inconsistent error.

What you expected to happen
When I try to change a user password it returns:old password is incorrect

How to reproduce it (as minimally and precisely as possible)

resp2, err2 := client.CreateUser(context.TODO(), []byte(`test`), []byte(`1Password!*`), auth.PermissionRW, "test")
err:= client.ChangePassword(context.TODO(), []byte(`test`), []byte(`1Password!*`), []byte(`2Password!*`))

Environment
immudb 0.6.2
Commit : 035845e
Built by: [email protected]
Built at: Thu, 02 Jul 2020 10:25:28 CEST

Additional info (any other context about the problem)

Please remember to fix related test inside client_test.go
@mmeloni mmeloni added bug Something isn't working Component::immudb Server component of immudb labels Jul 3, 2020
@padurean padurean added the Feature::multidb Multi database support label Jul 27, 2020
@vchain-us-mgmt vchain-us-mgmt added this to the v0.7 milestone Jul 27, 2020
@vchain-us-mgmt
Copy link
Contributor

Retesting with latest codebase needed

mmeloni added a commit that referenced this issue Jul 31, 2020
@mmeloni
chore: improve command ux and fix changepassword test. Closes #370
3bd0c54 
Signed-off-by: Michele Meloni <[email protected]>
mmeloni added a commit that referenced this issue Jul 31, 2020
@mmeloni
chore: improve command ux and fix changepassword test. Closes #370
8c32ddf 
Signed-off-by: Michele Meloni <[email protected]>
@mmeloni mmeloni linked a pull request Jul 31, 2020 that will close this issue
improving error handling #438
Merged
mmeloni added a commit that referenced this issue Aug 2, 2020
@mmeloni
chore: improve command ux and fix changepassword test. Closes #370
134de3b 
Signed-off-by: Michele Meloni <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working Component::immudb Server component of immudb Feature::multidb Multi database support
Projects
None yet
Milestone
v0.7
Development

Successfully merging a pull request may close this issue.

improving error handling codenotary/immudb
3 participants
@padurean @mmeloni @vchain-us-mgmt