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

User Manager: Changing password and adding host at the same time results in query error #2427

Closed
dmoagx opened this Issue Feb 28, 2016 · 2 comments

Comments

Projects
None yet
2 participants
@dmoagx
Member

dmoagx commented Feb 28, 2016

When adding a new host to a user SP will use this query (where password is copied from an existing user):

CREATE USER 'horst'@'%' IDENTIFIED BY PASSWORD '*27193CCC36970235099CF910EC3128B90A4EB4D4';

When changing a pasword the query is:

SET PASSWORD FOR 'horst'@'%' = PASSWORD('test');

However, when both adding a new host and changing the password at the same time the query is:

CREATE USER 'horst'@'localhost' IDENTIFIED BY PASSWORD 'foobar';

This is invalid and will be rejected by MySQL with Password hash should be a 41-digit hexadecimal number (or similar).

@dmoagx dmoagx added the Bug label Feb 28, 2016

@dmoagx dmoagx changed the title from User Manager: Changing password and adding host at the same time results in error to User Manager: Changing password and adding host at the same time results in query error Feb 28, 2016

dmoagx added a commit that referenced this issue Mar 3, 2016

dmoagx added a commit that referenced this issue Mar 3, 2016

@dmoagx dmoagx added this to the 1.1.2 milestone Mar 6, 2016

@abhibeckert

This comment has been minimized.

Show comment
Hide comment
@abhibeckert

abhibeckert Jun 7, 2016

Member

@dmoagx can you confirm this is fixed close the issue?

Member

abhibeckert commented Jun 7, 2016

@dmoagx can you confirm this is fixed close the issue?

@dmoagx

This comment has been minimized.

Show comment
Hide comment
@dmoagx

dmoagx Jul 9, 2016

Member

This has been included in 1.1.2

Member

dmoagx commented Jul 9, 2016

This has been included in 1.1.2

@dmoagx dmoagx closed this Jul 9, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment