Author: slaws
Date: Mon Sep 19 11:49:28 2011
New Revision: 1172578

URL: http://svn.apache.org/viewvc?rev=1172578&view=rev
Log:
TUSCANY-3949 = Applying Vijai's patch to correct policy namespaces + some extra 
fixes to correct the associated test case. 

Modified:
    
tuscany/sca-java-2.x/trunk/modules/policy-security/src/main/java/org/apache/tuscany/sca/policy/authorization/AuthorizationPolicy.java
    
tuscany/sca-java-2.x/trunk/modules/policy-security/src/main/resources/META-INF/services/org.apache.tuscany.sca.contribution.processor.StAXArtifactProcessor
    
tuscany/sca-java-2.x/trunk/modules/policy-security/src/test/java/org/apache/tuscany/sca/policy/security/PolicyProcessorTestCase.java
    
tuscany/sca-java-2.x/trunk/modules/policy-security/src/test/resources/org/apache/tuscany/sca/policy/security/mock_policy_definitions.xml

Modified: 
tuscany/sca-java-2.x/trunk/modules/policy-security/src/main/java/org/apache/tuscany/sca/policy/authorization/AuthorizationPolicy.java
URL: 
http://svn.apache.org/viewvc/tuscany/sca-java-2.x/trunk/modules/policy-security/src/main/java/org/apache/tuscany/sca/policy/authorization/AuthorizationPolicy.java?rev=1172578&r1=1172577&r2=1172578&view=diff
==============================================================================
--- 
tuscany/sca-java-2.x/trunk/modules/policy-security/src/main/java/org/apache/tuscany/sca/policy/authorization/AuthorizationPolicy.java
 (original)
+++ 
tuscany/sca-java-2.x/trunk/modules/policy-security/src/main/java/org/apache/tuscany/sca/policy/authorization/AuthorizationPolicy.java
 Mon Sep 19 11:49:28 2011
@@ -24,15 +24,15 @@ import java.util.List;
 
 import javax.xml.namespace.QName;
 
+import org.apache.tuscany.sca.assembly.xml.Constants;
+
 /**
  * Models the SCA Implementation Security Policy Assertion for Authorization.
  * 
  * @version $Rev$ $Date$
  */
 public class AuthorizationPolicy {
-    private final static String SCA11_NS = 
"http://docs.oasis-open.org/ns/opencsa/sca/200912";;
-    // private final static String SCA10_TUSCANY_NS = 
"http://tuscany.apache.org/xmlns/sca/1.1";;
-    public static final QName NAME = new QName(SCA11_NS, "authorization");
+    public static final QName NAME = new QName(Constants.SCA11_TUSCANY_NS, 
"authorization");
 
     public static enum AcessControl {
         permitAll, denyAll, allow

Modified: 
tuscany/sca-java-2.x/trunk/modules/policy-security/src/main/resources/META-INF/services/org.apache.tuscany.sca.contribution.processor.StAXArtifactProcessor
URL: 
http://svn.apache.org/viewvc/tuscany/sca-java-2.x/trunk/modules/policy-security/src/main/resources/META-INF/services/org.apache.tuscany.sca.contribution.processor.StAXArtifactProcessor?rev=1172578&r1=1172577&r2=1172578&view=diff
==============================================================================
--- 
tuscany/sca-java-2.x/trunk/modules/policy-security/src/main/resources/META-INF/services/org.apache.tuscany.sca.contribution.processor.StAXArtifactProcessor
 (original)
+++ 
tuscany/sca-java-2.x/trunk/modules/policy-security/src/main/resources/META-INF/services/org.apache.tuscany.sca.contribution.processor.StAXArtifactProcessor
 Mon Sep 19 11:49:28 2011
@@ -18,12 +18,10 @@
 # Implementation class for the artifact processor extension
 
org.apache.tuscany.sca.policy.authentication.basic.BasicAuthenticationPolicyProcessor;qname=http://tuscany.apache.org/xmlns/sca/1.1#basicAuthentication,model=org.apache.tuscany.sca.policy.authentication.basic.BasicAuthenticationPolicy
 
org.apache.tuscany.sca.policy.security.http.ssl.HTTPSPolicyProcessor;qname=http://tuscany.apache.org/xmlns/sca/1.1#https,model=org.apache.tuscany.sca.policy.security.http.ssl.HTTPSPolicy
-
-# TODO - following processors are old and somewhat questionable for 2.x, needs 
reviewing
-org.apache.tuscany.sca.policy.authorization.AuthorizationPolicyProcessor;qname=http://docs.oasis-open.org/ns/opencsa/sca/200912#authorization,model=org.apache.tuscany.sca.policy.authorization.AuthorizationPolicy
-org.apache.tuscany.sca.policy.authorization.AuthorizationPolicyProcessor;qname=http://docs.oasis-open.org/ns/opencsa/sca/200912#allow,model=org.apache.tuscany.sca.policy.authorization.AuthorizationPolicy
-org.apache.tuscany.sca.policy.authorization.AuthorizationPolicyProcessor;qname=http://docs.oasis-open.org/ns/opencsa/sca/200912#permitAll,model=org.apache.tuscany.sca.policy.authorization.AuthorizationPolicy
-org.apache.tuscany.sca.policy.authorization.AuthorizationPolicyProcessor;qname=http://docs.oasis-open.org/ns/opencsa/sca/200912#denyAll,model=org.apache.tuscany.sca.policy.authorization.AuthorizationPolicy
-org.apache.tuscany.sca.policy.identity.SecurityIdentityPolicyProcessor;qname=http://docs.oasis-open.org/ns/opencsa/sca/200912#runAs,model=org.apache.tuscany.sca.policy.identity.SecurityIdentityPolicy
+org.apache.tuscany.sca.policy.authorization.AuthorizationPolicyProcessor;qname=http://tuscany.apache.org/xmlns/sca/1.1#authorization,model=org.apache.tuscany.sca.policy.authorization.AuthorizationPolicy
+org.apache.tuscany.sca.policy.authorization.AuthorizationPolicyProcessor;qname=http://tuscany.apache.org/xmlns/sca/1.1#allow,model=org.apache.tuscany.sca.policy.authorization.AuthorizationPolicy
+org.apache.tuscany.sca.policy.authorization.AuthorizationPolicyProcessor;qname=http://tuscany.apache.org/xmlns/sca/1.1#permitAll,model=org.apache.tuscany.sca.policy.authorization.AuthorizationPolicy
+org.apache.tuscany.sca.policy.authorization.AuthorizationPolicyProcessor;qname=http://tuscany.apache.org/xmlns/sca/1.1#denyAll,model=org.apache.tuscany.sca.policy.authorization.AuthorizationPolicy
+org.apache.tuscany.sca.policy.identity.SecurityIdentityPolicyProcessor;qname=http://tuscany.apache.org/xmlns/sca/1.1#runAs,model=org.apache.tuscany.sca.policy.identity.SecurityIdentityPolicy
 
org.apache.tuscany.sca.policy.identity.SecurityIdentityPolicyProcessor;qname=http://tuscany.apache.org/xmlns/sca/1.1#securityIdentity,model=org.apache.tuscany.sca.policy.identity.SecurityIdentityPolicy
 
org.apache.tuscany.sca.policy.security.jaas.JaasAuthenticationPolicyProcessor;qname=http://tuscany.apache.org/xmlns/sca/1.1#jaasAuthentication,model=org.apache.tuscany.sca.policy.security.jaas.JaasAuthenticationPolicy

Modified: 
tuscany/sca-java-2.x/trunk/modules/policy-security/src/test/java/org/apache/tuscany/sca/policy/security/PolicyProcessorTestCase.java
URL: 
http://svn.apache.org/viewvc/tuscany/sca-java-2.x/trunk/modules/policy-security/src/test/java/org/apache/tuscany/sca/policy/security/PolicyProcessorTestCase.java?rev=1172578&r1=1172577&r2=1172578&view=diff
==============================================================================
--- 
tuscany/sca-java-2.x/trunk/modules/policy-security/src/test/java/org/apache/tuscany/sca/policy/security/PolicyProcessorTestCase.java
 (original)
+++ 
tuscany/sca-java-2.x/trunk/modules/policy-security/src/test/java/org/apache/tuscany/sca/policy/security/PolicyProcessorTestCase.java
 Mon Sep 19 11:49:28 2011
@@ -31,6 +31,7 @@ import javax.xml.stream.XMLInputFactory;
 import javax.xml.stream.XMLStreamConstants;
 import javax.xml.stream.XMLStreamReader;
 
+import org.apache.tuscany.sca.assembly.xml.Constants;
 import org.apache.tuscany.sca.contribution.processor.ProcessorContext;
 import org.apache.tuscany.sca.contribution.processor.StAXArtifactProcessor;
 import org.apache.tuscany.sca.policy.authorization.AuthorizationPolicy;
@@ -44,7 +45,7 @@ import org.junit.Test;
  * @version $Rev$ $Date$
  */
 public class PolicyProcessorTestCase {
-    private final static String SCA11_NS = 
"http://docs.oasis-open.org/ns/opencsa/sca/200912";;
+   
     private final static List<String> SEQ =
         Arrays.asList("permitAll",
                       "allow [r1, r2]",
@@ -62,10 +63,10 @@ public class PolicyProcessorTestCase {
         Map<QName, StAXArtifactProcessor> processors = new HashMap<QName, 
StAXArtifactProcessor>();
         processors.put(AuthorizationPolicy.NAME, new 
AuthorizationPolicyProcessor(null));
         processors.put(SecurityIdentityPolicy.NAME, new 
SecurityIdentityPolicyProcessor(null));
-        processors.put(new QName(SCA11_NS, "allow"), new 
AuthorizationPolicyProcessor(null));
-        processors.put(new QName(SCA11_NS, "permitAll"), new 
AuthorizationPolicyProcessor(null));
-        processors.put(new QName(SCA11_NS, "denyAll"), new 
AuthorizationPolicyProcessor(null));
-        processors.put(new QName(SCA11_NS, "runAs"), new 
SecurityIdentityPolicyProcessor(null));
+        processors.put(new QName(Constants.SCA11_TUSCANY_NS, "allow"), new 
AuthorizationPolicyProcessor(null));
+        processors.put(new QName(Constants.SCA11_TUSCANY_NS, "permitAll"), new 
AuthorizationPolicyProcessor(null));
+        processors.put(new QName(Constants.SCA11_TUSCANY_NS, "denyAll"), new 
AuthorizationPolicyProcessor(null));
+        processors.put(new QName(Constants.SCA11_TUSCANY_NS, "runAs"), new 
SecurityIdentityPolicyProcessor(null));
         InputStream is = 
getClass().getResourceAsStream("mock_policy_definitions.xml");
         XMLInputFactory factory = XMLInputFactory.newInstance();
         XMLStreamReader reader = factory.createXMLStreamReader(is);

Modified: 
tuscany/sca-java-2.x/trunk/modules/policy-security/src/test/resources/org/apache/tuscany/sca/policy/security/mock_policy_definitions.xml
URL: 
http://svn.apache.org/viewvc/tuscany/sca-java-2.x/trunk/modules/policy-security/src/test/resources/org/apache/tuscany/sca/policy/security/mock_policy_definitions.xml?rev=1172578&r1=1172577&r2=1172578&view=diff
==============================================================================
--- 
tuscany/sca-java-2.x/trunk/modules/policy-security/src/test/resources/org/apache/tuscany/sca/policy/security/mock_policy_definitions.xml
 (original)
+++ 
tuscany/sca-java-2.x/trunk/modules/policy-security/src/test/resources/org/apache/tuscany/sca/policy/security/mock_policy_definitions.xml
 Mon Sep 19 11:49:28 2011
@@ -24,23 +24,23 @@
 
     <!-- POLICY SETS -->
     <policySet name="ps1" provides="authorization" 
appliesTo="sca:implementation">
-        <authorization>
-            <permitAll />
-        </authorization>
+        <tuscany:authorization>
+            <tuscany:permitAll />
+        </tuscany:authorization>
     </policySet>
 
     <!-- POLICY SETS -->
     <policySet name="ps2" provides="authorization" 
appliesTo="sca:implementation">
-        <authorization>
-            <allow roles="r1 r2" />
-        </authorization>
+        <tuscany:authorization>
+            <tuscany:allow roles="r1 r2" />
+        </tuscany:authorization>
     </policySet>
 
     <!-- POLICY SETS -->
     <policySet name="ps3" provides="authorization" 
appliesTo="sca:implementation">
-        <authorization>
-            <denyAll />
-        </authorization>
+        <tuscany:authorization>
+            <tuscany:denyAll />
+        </tuscany:authorization>
     </policySet>
 
     <!-- POLICY SETS -->
@@ -59,22 +59,22 @@
 
     <!-- POLICY SETS -->
     <policySet name="ps6" provides="authorization" 
appliesTo="sca:implementation">
-        <permitAll />
+        <tuscany:permitAll />
     </policySet>
 
     <!-- POLICY SETS -->
     <policySet name="ps7" provides="authorization" 
appliesTo="sca:implementation">
-        <allow roles="r1 r2" />
+        <tuscany:allow roles="r1 r2" />
     </policySet>
 
     <!-- POLICY SETS -->
     <policySet name="ps8" provides="authorization" 
appliesTo="sca:implementation">
-        <denyAll />
+        <tuscany:denyAll />
     </policySet>
 
     <!-- POLICY SETS -->
     <policySet name="ps9" provides="securityIdentity" 
appliesTo="sca:implementation">
-        <runAs role="admin" />
+        <tuscany:runAs role="admin" />
     </policySet>
 
 </definitions>
\ No newline at end of file


Reply via email to