geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From djen...@apache.org
Subject svn commit: r109827 - geronimo/branches/djencks/jetty-deployer1/trunk/modules/security/src/java/org/apache/geronimo/security/jacc
Date Sat, 04 Dec 2004 21:40:29 GMT
Author: djencks
Date: Sat Dec  4 13:40:28 2004
New Revision: 109827

URL: http://svn.apache.org/viewcvs?view=rev&rev=109827
Log:
fix for GERONIMO-509
Modified:
   geronimo/branches/djencks/jetty-deployer1/trunk/modules/security/src/java/org/apache/geronimo/security/jacc/GeronimoPolicyConfigurationFactory.java
   geronimo/branches/djencks/jetty-deployer1/trunk/modules/security/src/java/org/apache/geronimo/security/jacc/PolicyConfigurationGeneric.java

Modified: geronimo/branches/djencks/jetty-deployer1/trunk/modules/security/src/java/org/apache/geronimo/security/jacc/GeronimoPolicyConfigurationFactory.java
Url: http://svn.apache.org/viewcvs/geronimo/branches/djencks/jetty-deployer1/trunk/modules/security/src/java/org/apache/geronimo/security/jacc/GeronimoPolicyConfigurationFactory.java?view=diff&rev=109827&p1=geronimo/branches/djencks/jetty-deployer1/trunk/modules/security/src/java/org/apache/geronimo/security/jacc/GeronimoPolicyConfigurationFactory.java&r1=109826&p2=geronimo/branches/djencks/jetty-deployer1/trunk/modules/security/src/java/org/apache/geronimo/security/jacc/GeronimoPolicyConfigurationFactory.java&r2=109827
==============================================================================
--- geronimo/branches/djencks/jetty-deployer1/trunk/modules/security/src/java/org/apache/geronimo/security/jacc/GeronimoPolicyConfigurationFactory.java
(original)
+++ geronimo/branches/djencks/jetty-deployer1/trunk/modules/security/src/java/org/apache/geronimo/security/jacc/GeronimoPolicyConfigurationFactory.java
Sat Dec  4 13:40:28 2004
@@ -43,11 +43,13 @@
     }
 
     public PolicyConfiguration getPolicyConfiguration(String contextID, boolean remove) throws
PolicyContextException {
-        PolicyConfiguration configuration = (PolicyConfiguration) configurations.get(contextID);
+        PolicyConfigurationGeneric configuration = (PolicyConfigurationGeneric) configurations.get(contextID);
 
         if (configuration == null || remove) {
             configuration = new PolicyConfigurationGeneric(contextID);
             configurations.put(contextID, configuration);
+        } else {
+            configuration.open();
         }
 
         return configuration;

Modified: geronimo/branches/djencks/jetty-deployer1/trunk/modules/security/src/java/org/apache/geronimo/security/jacc/PolicyConfigurationGeneric.java
Url: http://svn.apache.org/viewcvs/geronimo/branches/djencks/jetty-deployer1/trunk/modules/security/src/java/org/apache/geronimo/security/jacc/PolicyConfigurationGeneric.java?view=diff&rev=109827&p1=geronimo/branches/djencks/jetty-deployer1/trunk/modules/security/src/java/org/apache/geronimo/security/jacc/PolicyConfigurationGeneric.java&r1=109826&p2=geronimo/branches/djencks/jetty-deployer1/trunk/modules/security/src/java/org/apache/geronimo/security/jacc/PolicyConfigurationGeneric.java&r2=109827
==============================================================================
--- geronimo/branches/djencks/jetty-deployer1/trunk/modules/security/src/java/org/apache/geronimo/security/jacc/PolicyConfigurationGeneric.java
(original)
+++ geronimo/branches/djencks/jetty-deployer1/trunk/modules/security/src/java/org/apache/geronimo/security/jacc/PolicyConfigurationGeneric.java
Sat Dec  4 13:40:28 2004
@@ -220,4 +220,10 @@
     public boolean inService() throws PolicyContextException {
         return (state == IN_SERVICE);
     }
+
+    //TODO I have no idea what side effects this might have, but it's needed in some form
from GeronimoPolicyConfigurationFactory.
+    //see JACC spec 1.0 section 3.1.1.1 discussion of in service and deleted.
+    public void open() {
+        state = OPEN;
+    }
 }

Mime
View raw message