[java-identity-provider COMMIT] in /trunk/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl: ExtractRemoteUs...

[noreply at shibboleth.net]

Author: scantor
Date: Wed May 28 12:35:29 2014
New Revision: 5982

URL: http://svn.shibboleth.net/view/java-identity-provider?rev=5982&view=rev
Log:
Log cleanup

Modified:
    trunk/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/ExtractRemoteUser.java
    trunk/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/ExtractUserAgentAddress.java
    trunk/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/ExtractUserAgentIdentifier.java
    trunk/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/ExtractUsernamePasswordFromBasicAuth.java
    trunk/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/ExtractUsernamePasswordFromWssToken.java
    trunk/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/FilterFlowsByForcedAuthn.java
    trunk/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/FilterFlowsByPassivity.java
    trunk/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/SelectAuthenticationFlow.java
    trunk/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/ValidateRemoteUser.java
    trunk/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/ValidateUserAgentAddress.java
    trunk/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/ValidateUsernamePasswordAgainstKerberos.java

Modified: trunk/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/ExtractRemoteUser.java
URL: http://svn.shibboleth.net/view/java-identity-provider/trunk/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/ExtractRemoteUser.java?rev=5982&r1=5981&r2=5982&view=diff
==============================================================================
--- trunk/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/ExtractRemoteUser.java (original)
+++ trunk/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/ExtractRemoteUser.java Wed May 28 12:35:29 2014
@@ -76,7 +76,7 @@
      * 
      * @param flag value to set  
      */
-    public void setCheckRemoteUser(boolean flag) {
+    public void setCheckRemoteUser(final boolean flag) {
         ComponentSupport.ifInitializedThrowUnmodifiabledComponentException(this);
         
         checkRemoteUser = flag;
@@ -110,13 +110,12 @@
         super.doInitialize();
         
         if (!checkRemoteUser && checkAttributes.isEmpty() && checkHeaders.isEmpty()) {
-            log.debug("{} configuration contains no headers or attributes to check", getLogPrefix());
+            log.debug("{} Configuration contains no headers or attributes to check", getLogPrefix());
             throw new ComponentInitializationException("ExtractRemoteUser action configuration is invalid");
         }
     }
     
-    // Checkstyle: CyclomaticComplexity OFF
-    
+// Checkstyle: CyclomaticComplexity OFF
     /** {@inheritDoc} */
     @Override
     protected void doExecute(@Nonnull final ProfileRequestContext profileRequestContext,
@@ -124,7 +123,7 @@
 
         final HttpServletRequest request = getHttpServletRequest();
         if (request == null) {
-            log.debug("{} profile action does not contain an HttpServletRequest", getLogPrefix());
+            log.debug("{} Profile action does not contain an HttpServletRequest", getLogPrefix());
             ActionSupport.buildEvent(profileRequestContext, AuthnEventIds.NO_CREDENTIALS);
             return;
         }
@@ -133,7 +132,7 @@
         if (checkRemoteUser) {
             username = request.getRemoteUser();
             if (username != null && !username.isEmpty()) {
-                log.debug("{} user identity extracted from REMOTE_USER: {}", getLogPrefix(), username);
+                log.debug("{} User identity extracted from REMOTE_USER: {}", getLogPrefix(), username);
                 authenticationContext.getSubcontext(UsernameContext.class, true).setUsername(
                         applyTransforms(username));
                 return;
@@ -143,7 +142,7 @@
         for (String s : checkAttributes) {
             Object attr = request.getAttribute(s);
             if (attr != null && !attr.toString().isEmpty()) {
-                log.debug("{} user identity extracted from attribute {}: {}", getLogPrefix(), s, attr);
+                log.debug("{} User identity extracted from attribute {}: {}", getLogPrefix(), s, attr);
                 authenticationContext.getSubcontext(UsernameContext.class, true).setUsername(
                         applyTransforms(attr.toString()));
                 return;
@@ -153,17 +152,16 @@
         for (String s : checkHeaders) {
             username = request.getHeader(s);
             if (username != null && !username.isEmpty()) {
-                log.debug("{} user identity extracted from header {}: {}", getLogPrefix(), s, username);
+                log.debug("{} User identity extracted from header {}: {}", getLogPrefix(), s, username);
                 authenticationContext.getSubcontext(UsernameContext.class, true).setUsername(
                         applyTransforms(username));
                 return;
             }
         }
         
-        log.debug("{} no user identity found in request", getLogPrefix());

[... 423 lines stripped ...]