fix(health): token_broker health check only when tokens are enabled #72

2026-11-09T18:24:47Z

james.maes commented

2026-11-09 18:24:47 +00:00

Registers the token_broker health check only when tokens are enabled (AuthentikQBitConfig.getTokensEnabled()). With tokens off it could only show UNKNOWN 'not configured' forever (core cannot remove a registered check). Other checks unchanged.

Tests: registrationWithTokensOffOmitsTokenBroker (failed before, passes after), registrationWithTokensOnKeepsTokenBroker. Local: ./mvnw verify -DskipITs, 626 tests green, Checkstyle/SpotBugs/PMD gates passed; contract ITs run in CI.

Registers the token_broker health check only when tokens are enabled (AuthentikQBitConfig.getTokensEnabled()). With tokens off it could only show UNKNOWN 'not configured' forever (core cannot remove a registered check). Other checks unchanged. Tests: registrationWithTokensOffOmitsTokenBroker (failed before, passes after), registrationWithTokensOnKeepsTokenBroker. Local: ./mvnw verify -DskipITs, 626 tests green, Checkstyle/SpotBugs/PMD gates passed; contract ITs run in CI.

james.maes added 1 commit 2026-11-09 18:24:48 +00:00

fix(health): register the token_broker health check only when tokens are enabled

maven / build (push) Successful in 11m34s

Details

build / maven (push) Successful in 11m34s

Details

82c79d69be

With AUTHENTIK_QBIT_TOKENS off the check could only ever report UNKNOWN
"not configured", and core has no way to remove a registered check, so the
health page showed a permanent UNKNOWN row. Register it only when
getTokensEnabled() is true; the other checks are unchanged.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

james.maes merged commit a3fa59efe7 into develop

2026-11-09 18:46:34 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: QRun-IO/qbit-authentik#72