Fire OIDC event when server not available

quarkusio · Dec 16, 2023 · 8811efc · 8811efc
1 parent 1c5af65
commit 8811efc
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/docs/src/main/asciidoc/security-customization.adoc b/docs/src/main/asciidoc/security-customization.adoc
@@ -669,7 +669,7 @@ public class SecurityEventObserver {
     void observeAuthorizationSuccess(@ObservesAsync AuthorizationSuccessEvent event) {
         String principalName = getPrincipalName(event);
         if (principalName != null) {
-            LOG.debugf("User '%s' has been authorized successfully", event.getSecurityIdentity().getPrincipal().getName());
+            LOG.debugf("User '%s' has been authorized successfully", principalName);
         }
     }
 

diff --git a/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/runtime/OidcRecorder.java b/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/runtime/OidcRecorder.java
@@ -495,12 +495,11 @@ public OidcConfigurationMetadata apply(JsonObject json) {
 
                     @Override
                     public Uni<OidcProviderClient> apply(OidcConfigurationMetadata metadata, Throwable t) {
+                        String tenantId = oidcConfig.tenantId.orElse(DEFAULT_TENANT_ID);
                         if (t != null) {
                             client.close();
-                            return Uni.createFrom().failure(
-                                    toOidcException(t, authServerUriString, oidcConfig.tenantId.orElse(DEFAULT_TENANT_ID)));
+                            return Uni.createFrom().failure(toOidcException(t, authServerUriString, tenantId));
                         }
-                        String tenantId = oidcConfig.tenantId.orElse(DEFAULT_TENANT_ID);
                         if (shouldFireOidcServerAvailableEvent(tenantId)) {
                             fireOidcServerAvailableEvent(authServerUriString, tenantId);
                         }