cxf-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From cohei...@apache.org
Subject [2/2] cxf-fediz git commit: Fixing some checkstyle issues
Date Thu, 05 Mar 2015 17:39:14 GMT
Fixing some checkstyle issues


Project: http://git-wip-us.apache.org/repos/asf/cxf-fediz/repo
Commit: http://git-wip-us.apache.org/repos/asf/cxf-fediz/commit/f49b28e6
Tree: http://git-wip-us.apache.org/repos/asf/cxf-fediz/tree/f49b28e6
Diff: http://git-wip-us.apache.org/repos/asf/cxf-fediz/diff/f49b28e6

Branch: refs/heads/master
Commit: f49b28e67dfc12943442fa0ed67528c488ad8ef5
Parents: 23cc34f
Author: Colm O hEigeartaigh <coheigea@apache.org>
Authored: Thu Mar 5 17:38:22 2015 +0000
Committer: Colm O hEigeartaigh <coheigea@apache.org>
Committed: Thu Mar 5 17:38:22 2015 +0000

----------------------------------------------------------------------
 .../apache/cxf/fediz/service/idp/STSAuthenticationProvider.java  | 4 ++--
 .../apache/cxf/fediz/service/sts/FedizX509DelegationHandler.java | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cxf-fediz/blob/f49b28e6/services/idp/src/main/java/org/apache/cxf/fediz/service/idp/STSAuthenticationProvider.java
----------------------------------------------------------------------
diff --git a/services/idp/src/main/java/org/apache/cxf/fediz/service/idp/STSAuthenticationProvider.java
b/services/idp/src/main/java/org/apache/cxf/fediz/service/idp/STSAuthenticationProvider.java
index bdd7727..3b4be6a 100644
--- a/services/idp/src/main/java/org/apache/cxf/fediz/service/idp/STSAuthenticationProvider.java
+++ b/services/idp/src/main/java/org/apache/cxf/fediz/service/idp/STSAuthenticationProvider.java
@@ -30,7 +30,6 @@ import java.util.Map;
 
 import javax.security.auth.callback.CallbackHandler;
 import javax.security.auth.login.LoginException;
-import javax.servlet.http.HttpServletRequest;
 import javax.xml.namespace.QName;
 
 import org.w3c.dom.Document;
@@ -43,7 +42,6 @@ import org.apache.cxf.fediz.core.ClaimTypes;
 import org.apache.cxf.fediz.service.idp.kerberos.KerberosServiceRequestToken;
 import org.apache.cxf.fediz.service.idp.kerberos.KerberosTokenValidator;
 import org.apache.cxf.fediz.service.idp.kerberos.PassThroughKerberosClient;
-import org.apache.cxf.fediz.service.idp.util.WebUtils;
 import org.apache.cxf.helpers.DOMUtils;
 //import org.apache.cxf.transport.http.HTTPConduit;
 //import org.apache.cxf.transports.http.configuration.HTTPClientPolicy;
@@ -121,6 +119,7 @@ public class STSAuthenticationProvider implements AuthenticationProvider
{
     private String technicalPassword;
     
     
+    //CHECKSTYLE:OFF
     @Override
     public Authentication authenticate(Authentication authentication) throws AuthenticationException
{
         
@@ -274,6 +273,7 @@ public class STSAuthenticationProvider implements AuthenticationProvider
{
         }
         
     }
+    //CHECKSTYLE:ON
     
     private Principal validateKerberosToken(
         KerberosServiceRequestToken token,

http://git-wip-us.apache.org/repos/asf/cxf-fediz/blob/f49b28e6/services/sts/src/main/java/org/apache/cxf/fediz/service/sts/FedizX509DelegationHandler.java
----------------------------------------------------------------------
diff --git a/services/sts/src/main/java/org/apache/cxf/fediz/service/sts/FedizX509DelegationHandler.java
b/services/sts/src/main/java/org/apache/cxf/fediz/service/sts/FedizX509DelegationHandler.java
index a597cf3..38c19b2 100644
--- a/services/sts/src/main/java/org/apache/cxf/fediz/service/sts/FedizX509DelegationHandler.java
+++ b/services/sts/src/main/java/org/apache/cxf/fediz/service/sts/FedizX509DelegationHandler.java
@@ -37,7 +37,7 @@ public class FedizX509DelegationHandler implements TokenDelegationHandler
{
             Element tokenElement = (Element)token;
             String namespace = tokenElement.getNamespaceURI();
             String localname = tokenElement.getLocalName();
-            if ((WSConstants.SIG_NS.equals(namespace) && WSConstants.X509_DATA_LN.equals(localname)))
{
+            if (WSConstants.SIG_NS.equals(namespace) && WSConstants.X509_DATA_LN.equals(localname))
{
                 return true;
             }
         }


Mime
View raw message