confluentinc / cp-ansible

Ansible playbooks for the Confluent Platform
Apache License 2.0
42 stars 405 forks source link

[ANSIENG-3807] | Fix controller super users by adding cert principal #1819

Closed rrbadiani closed 3 weeks ago

rrbadiani commented 3 weeks ago

Description

super.users in controller should contain the mtls cert's principal. Adding that and refactoring the code to handle multiple principals coming from ldap/oauth/cert.

Fixes # (issue)

Type of change

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Checklist:

confluent-cla-assistant[bot] commented 3 weeks ago

:tada: All Contributor License Agreements have been signed. Ready to merge.
Please push an empty commit if you would like to re-run the checks to verify CLA status for all contributors.