Merge pull request #19100 from sberyozkin/conflicting_auth_mechanisms

Update HttpCredentialTransport to keep a lower case authentication scheme
quarkusio · Jul 30, 2021 · d2a6c85 · d2a6c85
2 parents d8d04a1 + 7c63f4d
commit d2a6c85
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/...sions/oidc/runtime/src/main/java/io/quarkus/oidc/runtime/OidcAuthenticationMechanism.java b/...sions/oidc/runtime/src/main/java/io/quarkus/oidc/runtime/OidcAuthenticationMechanism.java
@@ -9,6 +9,7 @@
 
 import io.quarkus.oidc.OIDCException;
 import io.quarkus.oidc.OidcTenantConfig;
+import io.quarkus.oidc.common.runtime.OidcConstants;
 import io.quarkus.security.identity.IdentityProviderManager;
 import io.quarkus.security.identity.SecurityIdentity;
 import io.quarkus.security.identity.request.AuthenticationRequest;
@@ -79,6 +80,6 @@ public Set<Class<? extends AuthenticationRequest>> getCredentialTypes() {
     public HttpCredentialTransport getCredentialTransport() {
         //not 100% correct, but enough for now
         //if OIDC is present we don't really want another bearer mechanism
-        return new HttpCredentialTransport(HttpCredentialTransport.Type.AUTHORIZATION, "bearer");
+        return new HttpCredentialTransport(HttpCredentialTransport.Type.AUTHORIZATION, OidcConstants.BEARER_SCHEME);
     }
 }
diff --git a/...runtime/src/main/java/io/quarkus/vertx/http/runtime/security/HttpCredentialTransport.java b/...runtime/src/main/java/io/quarkus/vertx/http/runtime/security/HttpCredentialTransport.java
@@ -26,7 +26,7 @@ public HttpCredentialTransport(Type transportType, String typeTarget) {
     public HttpCredentialTransport(Type transportType, String typeTarget, String authenticationScheme) {
         this.transportType = Objects.requireNonNull(transportType);
         this.typeTarget = Objects.requireNonNull(typeTarget).toLowerCase();
-        this.authenticationScheme = Objects.requireNonNull(authenticationScheme);
+        this.authenticationScheme = Objects.requireNonNull(authenticationScheme).toLowerCase();
     }
 
     public enum Type {