keycloak_user_federation: fix diff of empty krbPrincipalAttribute

[yeoldegrove]

SUMMARY

krbPrincipalAttribute is supposed to have an empty string as a valid value.

community.general/plugins/modules/keycloak_user_federation.py

Line 346 in bc609d7

When this is empty, the LDAP user will be looked based on LDAP username corresponding

Without this bugfix, an ansible-playbook ... --check --diffrun will always show an empty string in the diff if you pass en empty string to config.krbPrincipalAttribute.

TASK [kc1 - realm-1 - test-2 - Create or update a Keycloak kerberos enabled user_federation] ********************************************************************************************************************************************
--- before
+++ after
@@ -17,6 +17,7 @@
         "importEnabled": "false",
         "kerberosRealm": "EXAMPLE.NET",
         "keyTab": "/mnt/ad2",
+        "krbPrincipalAttribute": "",
         "pagination": "false",
         "priority": "0",
         "rdnLDAPAttribute": "sAMAccountName",

ISSUE TYPE

• Bugfix Pull Request

COMPONENT NAME

keycloak_user_federation

[ansibullbot]

cc @eikef @laurpaum @mattock @ndclt
click here for bot help

[felixfontein]

Thanks for your contribution! The change looks good to me. If nobody objects, I'll merge it in ~a week.

[patchback]

Backport to stable-7: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-7/a71e19130d041752aa393be85c22adcb1ad63e7a/pr-8320

Backported as #8339

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

[patchback]

Backport to stable-8: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-8/a71e19130d041752aa393be85c22adcb1ad63e7a/pr-8320

Backported as #8340

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

[felixfontein]

@yeoldegrove thanks for your contribution!