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

2026-11-09T18:24:46Z

james.maes commented

2026-11-09 18:24:46 +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:46 +00:00

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

maven / build (push) Successful in 11m22s

Details

build / maven (push) Successful in 11m22s

Details

477bfff3a2

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 d9e969f3e1 into release/0.1

2026-11-09 18:36:25 +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#71