Skip to content

Commit

Permalink
Update quay.io/keycloak/keycloak Docker tag to v26 (#330)
Browse files Browse the repository at this point in the history
* Update quay.io/keycloak/keycloak Docker tag to v26

* Fix healthcheck for keycloak 26

The healthcheck was failing for two reasons:
1. --health-enabled now defaults to false 
2. It validates HTTP requests more strictly. Thus a bug in the GET
   request from healthcheck.sh was successful in 25.0 but gave a 400
   response in 26.0

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Spencer Bliven <spencer.bliven@gmail.com>
  • Loading branch information
renovate[bot] and sbliven authored Nov 21, 2024
1 parent 29ffc0c commit 2e3a690
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions services/backend/services/keycloak/compose.base.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
services:
keycloak:
image: quay.io/keycloak/keycloak:25.0
command: start-dev --import-realm --http-port=80
image: quay.io/keycloak/keycloak:26.0
command: start-dev --import-realm --health-enabled=true --http-port=80
env_file: ./config/.env
volumes:
- ./healthcheck/healthcheck.sh:/healthcheck.sh
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ if ! exec 3<>/dev/tcp/localhost/9000; then
exit 1
fi

printf "GET /health/ready HTTP/1.1\r\nHost: localhostConnection: close\r\n\r\n" >&3
printf "GET /health/ready HTTP/1.1\r\nHost: localhost\r\nConnection: close\r\n\r\n" >&3
timeout --preserve-status 5 grep -m 1 "UP" <&3
exit_status=$?
exec 3>&-
Expand Down

0 comments on commit 2e3a690

Please sign in to comment.