keycloak-aplcache

Update RequestAuthenticator.java

3/15/2017 5:13:41 AM

Details

diff --git a/adapters/oidc/adapter-core/src/main/java/org/keycloak/adapters/RequestAuthenticator.java b/adapters/oidc/adapter-core/src/main/java/org/keycloak/adapters/RequestAuthenticator.java
index d8869fd..158bc9d 100755
--- a/adapters/oidc/adapter-core/src/main/java/org/keycloak/adapters/RequestAuthenticator.java
+++ b/adapters/oidc/adapter-core/src/main/java/org/keycloak/adapters/RequestAuthenticator.java
@@ -160,22 +160,13 @@ public abstract class RequestAuthenticator {
     }
 
     protected boolean verifySSL() {
-        boolean verificationFail  = !facade.getRequest().isSecure()
-                && deployment.getSslRequired().isRequired(facade.getRequest().getRemoteAddr());
-        if (verificationFail){
-            final String remoteAddr = facade.getRequest().getRemoteAddr();
-            final SslRequired sslRequired = deployment.getSslRequired();
-            log.warnf("SSL is required to authenticate. "+
-                            "Passed: %s, request is secure: %s, " +
-                            "SSL is required for: %s, " +
-                            "SSL is required for remote addr %s: %s",
-                    !verificationFail,
-                    facade.getRequest().isSecure(),
-                    sslRequired.name(),
-                    remoteAddr,
-                    sslRequired.isRequired(remoteAddr));
-        }
-        return verificationFail;
+        if (!facade.getRequest().isSecure()
+                && deployment.getSslRequired().isRequired(facade.getRequest().getRemoteAddr())) {
+            log.warnf("SSL is required to authenticate. Remote address %s is secure: %s, SSL required for: %s .",
+                    facade.getRequest().getRemoteAddr(), facade.getRequest().isSecure(), deployment.getSslRequired().name());
+            return true;
+        }
+        return false;
     }
 
     protected boolean isAutodetectedBearerOnly(HttpFacade.Request request) {