You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tuscany.apache.org by sv...@apache.org on 2007/12/18 08:29:51 UTC

svn commit: r605112 - in /incubator/tuscany/java/sca/modules: implementation-java-runtime/src/main/java/org/apache/tuscany/sca/implementation/java/invocation/ policy-security/src/main/java/org/apache/tuscany/sca/policy/security/jaas/

Author: svkrish
Date: Mon Dec 17 23:29:50 2007
New Revision: 605112

URL: http://svn.apache.org/viewvc?rev=605112&view=rev
Log:
moving login config moudule class to calculator-implementation-samples

Removed:
    incubator/tuscany/java/sca/modules/policy-security/src/main/java/org/apache/tuscany/sca/policy/security/jaas/JaasLoginModule.java
Modified:
    incubator/tuscany/java/sca/modules/implementation-java-runtime/src/main/java/org/apache/tuscany/sca/implementation/java/invocation/PolicyHandlingInterceptor.java
    incubator/tuscany/java/sca/modules/policy-security/src/main/java/org/apache/tuscany/sca/policy/security/jaas/JaasAuthenticationPolicyHandler.java

Modified: incubator/tuscany/java/sca/modules/implementation-java-runtime/src/main/java/org/apache/tuscany/sca/implementation/java/invocation/PolicyHandlingInterceptor.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/java/sca/modules/implementation-java-runtime/src/main/java/org/apache/tuscany/sca/implementation/java/invocation/PolicyHandlingInterceptor.java?rev=605112&r1=605111&r2=605112&view=diff
==============================================================================
--- incubator/tuscany/java/sca/modules/implementation-java-runtime/src/main/java/org/apache/tuscany/sca/implementation/java/invocation/PolicyHandlingInterceptor.java (original)
+++ incubator/tuscany/java/sca/modules/implementation-java-runtime/src/main/java/org/apache/tuscany/sca/implementation/java/invocation/PolicyHandlingInterceptor.java Mon Dec 17 23:29:50 2007
@@ -45,7 +45,9 @@
             applyPreInvocationPolicies(targetOperation, msg);
             msg = next.invoke(msg);
         } finally {
-            applyPostInvocationPolices(targetOperation, msg);
+            if ( !msg.isFault() ) {
+                applyPostInvocationPolices(targetOperation, msg);
+            }
         }
         return msg;
     }
@@ -65,8 +67,8 @@
     }
 
     private void applyPostInvocationPolices(Object... context) {
-        for (PolicyHandler policyHandler : policyHandlers) {
-            policyHandler.afterInvoke(context);
+        for ( int count = policyHandlers.size() - 1 ; count >= 0 ; --count) {
+            policyHandlers.get(count).afterInvoke(context);
         }
     }
 }

Modified: incubator/tuscany/java/sca/modules/policy-security/src/main/java/org/apache/tuscany/sca/policy/security/jaas/JaasAuthenticationPolicyHandler.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/java/sca/modules/policy-security/src/main/java/org/apache/tuscany/sca/policy/security/jaas/JaasAuthenticationPolicyHandler.java?rev=605112&r1=605111&r2=605112&view=diff
==============================================================================
--- incubator/tuscany/java/sca/modules/policy-security/src/main/java/org/apache/tuscany/sca/policy/security/jaas/JaasAuthenticationPolicyHandler.java (original)
+++ incubator/tuscany/java/sca/modules/policy-security/src/main/java/org/apache/tuscany/sca/policy/security/jaas/JaasAuthenticationPolicyHandler.java Mon Dec 17 23:29:50 2007
@@ -46,8 +46,6 @@
     }
     
     public void beforeInvoke(Object... context) { 
-        Message msg = null;
-        
         try {
             CallbackHandler callbackHandler = (CallbackHandler)
                 Class.forName(((JaasAuthenticationPolicy)applicablePolicySet.getPolicies().get(0)).



---------------------------------------------------------------------
To unsubscribe, e-mail: tuscany-commits-unsubscribe@ws.apache.org
For additional commands, e-mail: tuscany-commits-help@ws.apache.org