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

keycloak_quarkus: enable config of key store and trust store #116

Merged
merged 1 commit into from
Nov 13, 2023

Conversation

Footur
Copy link
Contributor

@Footur Footur commented Oct 13, 2023

Breaking change: the parameter keycloak_quarkus_https_enabled of role keycloak_quarkus has been renamed to keycloak_quarkus_https_key_file_enabled.

New parameters:

Variable Description Default
keycloak_quarkus_key_store_file The file pat to the key store {{ keycloak.home }}/conf/key_store.p12
keycloak_quarkus_key_store_password Password for the key store ""
keycloak_quarkus_https_trust_store_enabled Enalbe confiugration of a trust store False
keycloak_quarkus_trust_store_file The file pat to the trust store {{ keycloak.home }}/conf/trust_store.p12
keycloak_quarkus_trust_store_password Password for the trust store ""

@Footur Footur force-pushed the keystore branch 2 times, most recently from b135c48 to e842462 Compare October 13, 2023 14:29
@guidograzioli
Copy link
Member

Looks good to me, thanks for the PR! I'll mark it "breaking change" because of the parameter renaming, and will need to edit the PR title and text with details after merging, for the changelog. The next release will be a major version bump, so no problem about it.

@guidograzioli guidograzioli added the breaking_changes MUST include changes that break existing playbooks label Oct 16, 2023
@Footur
Copy link
Contributor Author

Footur commented Oct 17, 2023

Cool, thanks for feedback. 👍

@guidograzioli guidograzioli merged commit cee02cf into ansible-middleware:main Nov 13, 2023
15 checks passed
@guidograzioli guidograzioli changed the title [keycloak_quarkus] Enable config of a key store and trust store keycloak_quarkus: enable config of key store and trust store Nov 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking_changes MUST include changes that break existing playbooks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants