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

[Backport][ipa-4-10] Fix 389-ds healthcheck output message #6509

Closed

Conversation

flo-renaud
Copy link
Contributor

This PR was opened automatically because PR #6450 was pushed to master and backport to ipa-4-10 is required.

…healthcheck.

Previously the message was:

"\n\nIn Directory Server, we offer one hash suitable for this "
"(PBKDF2_SHA256) and one hash\nfor \"legacy\" support (SSHA512)."
"\n\nYour configuration does not use these for password storage "
"or the root password storage\nscheme.\n"

but now the message is:

\n\nIn Directory Server, we offer one hash suitable for this "
"(PBKDF2-SHA512) and one hash\nfor \"legacy\" support (SSHA512)."
"\n\nYour configuration does not use these for password storage "
"or the root password storage\nscheme.\n"

PBKDF2_SHA256 has been replaced with PBKDF2-SHA512

Pagure: https://pagure.io/freeipa/issue/9238

Signed-off-by: Sumedh Sidhaye <ssidhaye@redhat.com>
@flo-renaud
Copy link
Contributor Author

PR was ACKed automatically because this is backport of PR #6450. Wait for CI to finish before pushing. In case of questions or problems contact @ssidhaye who is author of the original PR.

@flo-renaud flo-renaud added ack Pull Request approved, can be merged pushed Pull Request has already been pushed labels Oct 12, 2022
@flo-renaud
Copy link
Contributor Author

ipa-4-10:

  • 5477a07 With the commit #99a74d7, 389-ds changed the message returned in ipa-healthcheck.

@flo-renaud flo-renaud closed this Oct 12, 2022
@flo-renaud flo-renaud deleted the backport_pr6450_ipa-4-10 branch October 12, 2022 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
2 participants