You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uima.apache.org by ch...@apache.org on 2013/03/12 21:25:28 UTC

svn commit: r1455699 - /uima/sandbox/uima-ducc/trunk/uima-ducc-sm/src/main/java/org/apache/uima/ducc/sm/ServiceManagerComponent.java

Author: challngr
Date: Tue Mar 12 20:25:28 2013
New Revision: 1455699

URL: http://svn.apache.org/r1455699
Log:
UIMA-2687
Fix logic inversion checking user identity in SM.

Modified:
    uima/sandbox/uima-ducc/trunk/uima-ducc-sm/src/main/java/org/apache/uima/ducc/sm/ServiceManagerComponent.java

Modified: uima/sandbox/uima-ducc/trunk/uima-ducc-sm/src/main/java/org/apache/uima/ducc/sm/ServiceManagerComponent.java
URL: http://svn.apache.org/viewvc/uima/sandbox/uima-ducc/trunk/uima-ducc-sm/src/main/java/org/apache/uima/ducc/sm/ServiceManagerComponent.java?rev=1455699&r1=1455698&r2=1455699&view=diff
==============================================================================
--- uima/sandbox/uima-ducc/trunk/uima-ducc-sm/src/main/java/org/apache/uima/ducc/sm/ServiceManagerComponent.java (original)
+++ uima/sandbox/uima-ducc/trunk/uima-ducc-sm/src/main/java/org/apache/uima/ducc/sm/ServiceManagerComponent.java Tue Mar 12 20:25:28 2013
@@ -559,8 +559,10 @@ public class ServiceManagerComponent 
         byte[] auth_block= req.getAuth();
         boolean validated = false;
 
+        if ( ! signature_required ) return true;
+
         try {
-            validated = signature_required && check_signature(user, auth_block);
+            validated = check_signature(user, auth_block);
         } catch ( Throwable t ) {
             logger.error(methodName, null, "Crypto failure:", t.toString());
         }