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; + } }
