Skip to content

Commit

Permalink
Correctify health_url param
Browse files Browse the repository at this point in the history
It's empirical best practice to have a the relative path included in the configuration hostname configuration, too. (`hostname-debug` is a handy option for debugging this).

For example:

keycloak_quarkus_hostname: "https://sso-test.example.loc/auth"
keycloak_quarkus_http_relative_path: "/auth" # must start with `/`
  • Loading branch information
hwo-wd committed Dec 10, 2024
1 parent 76d6547 commit 3d1b42c
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions roles/keycloak_quarkus/vars/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,7 @@ keycloak: # noqa var-naming this is an internal dict of interpolated values
config_dir: "{{ keycloak_quarkus_config_dir }}"
bundle: "{{ keycloak_quarkus_archive }}"
service_name: "keycloak"
health_url: "{{ keycloak_quarkus_hostname }}{{ keycloak_quarkus_http_relative_path }}{{ '/' \
if keycloak_quarkus_http_relative_path | length > 1 else '' }}{{ keycloak_quarkus_health_check_url_path | default('realms/master/.well-known/openid-configuration') }}"
health_url: "{{ keycloak_quarkus_hostname }}/{{ keycloak_quarkus_health_check_url_path | default('realms/master/.well-known/openid-configuration') }}"
cli_path: "{{ keycloak_quarkus_home }}/bin/kcadm.sh"
service_user: "{{ keycloak_quarkus_service_user }}"
service_group: "{{ keycloak_quarkus_service_group }}"
Expand Down

0 comments on commit 3d1b42c

Please sign in to comment.