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

[PR #5934/78297e44 backport][stable-5] Fix keycloak sanitize_cr #6035

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Feb 22, 2023

This is a backport of PR #5934 as merged into main (78297e4).

SUMMARY

When we want to set the private saml key with the module keycloak_client keycloak receive the value no_log

The function sanitize_cr set the no_log value to avoid print the key value in the ansible log.
But the function copy of dict only copy the first level and keep the reference for sub dict.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

keycloak_client

* Fix keycloak sanitize_cr

* Update changelogs/fragments/5934-fix-keycloak-sanitize_cr.yml

Co-authored-by: Felix Fontein <[email protected]>

---------

Co-authored-by: Claude Dioudonnat <[email protected]>
Co-authored-by: Felix Fontein <[email protected]>
(cherry picked from commit 78297e4)
@patchback patchback bot mentioned this pull request Feb 22, 2023
@ansibullbot ansibullbot added backport bug This issue/PR relates to a bug module module new_contributor Help guide this first time contributor plugins plugin (any type) labels Feb 22, 2023
@felixfontein felixfontein merged commit 7ff20a5 into stable-5 Feb 22, 2023
@felixfontein felixfontein deleted the patchback/backports/stable-5/78297e44b74992c2108b6ffbb9b0b37e0d708c79/pr-5934 branch February 22, 2023 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug module module new_contributor Help guide this first time contributor plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants