Skip to content

add mysql_native_password plugin to authentication_string vs password #1441

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

Merged
merged 2 commits into from
Nov 15, 2021

Conversation

Heidistein
Copy link
Contributor

On a debian-11 server the plugin kept changing the password of a user. Turns out this user is using the plugin 'mysql_native_password', which does not use the PASSWORD field.

On a debian-11 server the plugin kept changing the password of a user. Turns out this user is using the plugin 'mysql_native_password', which does not use the PASSWORD field.
@Heidistein Heidistein requested a review from a team as a code owner September 24, 2021 14:31
@CLAassistant
Copy link

CLAassistant commented Sep 24, 2021

CLA assistant check
All committers have signed the CLA.

@Heidistein
Copy link
Contributor Author

Those failing tests are not my failures, it is RPM dependency related

@daianamezdrea
Copy link
Contributor

Hello @Heidistein, sorry for the late answer but our team went through some transformations, indeed the failures are not related with your PR, can you rebase it in order to have everything green and ready to merge? Thank you and sorry again

@Heidistein
Copy link
Contributor Author

I have rebased the tree. Thanks

@daianamezdrea
Copy link
Contributor

Hi @Heidistein, this looks good, the ubuntu errors are not related with your PR, thank you!

@daianamezdrea daianamezdrea merged commit 77cce4e into puppetlabs:main Nov 15, 2021
@Heidistein Heidistein deleted the patch-1 branch November 15, 2021 14:34
@Heidistein
Copy link
Contributor Author

Thanks!

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

Successfully merging this pull request may close these issues.

3 participants