geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From a..@apache.org
Subject svn commit: r123427 - /geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/jaas/GeronimoLoginConfiguration.java
Date Mon, 27 Dec 2004 16:21:06 GMT
Author: adc
Date: Mon Dec 27 08:21:06 2004
New Revision: 123427

URL: http://svn.apache.org/viewcvs?view=rev&rev=123427
Log:
GeronimoLoginConfiguration does not properly cleanup
http://nagoya.apache.org/jira/browse/GERONIMO-532
Modified:
   geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/jaas/GeronimoLoginConfiguration.java

Modified: geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/jaas/GeronimoLoginConfiguration.java
Url: http://svn.apache.org/viewcvs/geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/jaas/GeronimoLoginConfiguration.java?view=diff&rev=123427&p1=geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/jaas/GeronimoLoginConfiguration.java&r1=123426&p2=geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/jaas/GeronimoLoginConfiguration.java&r2=123427
==============================================================================
--- geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/jaas/GeronimoLoginConfiguration.java
(original)
+++ geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/jaas/GeronimoLoginConfiguration.java
Mon Dec 27 08:21:06 2004
@@ -101,7 +101,7 @@
         ConfigurationEntryFactory factory = (ConfigurationEntryFactory) event.getMember();
 
         entries.remove(factory.getConfigurationName());
-        log.info("Removed ACE " + factory.getConfigurationName());
+        log.info("Removed Application Configuration Entry " + factory.getConfigurationName());
     }
 
     private final void addConfiguration(ConfigurationEntryFactory factory) {
@@ -115,7 +115,7 @@
         AppConfigurationEntry ace = new AppConfigurationEntry(config.getLoginModuleClassName(),
config.getFlag().getFlag(), config.getOptions());
 
         entries.put(factory.getConfigurationName(), ace);
-        log.info("Added ACE " + factory.getConfigurationName());
+        log.info("Added Application Configuration Entry " + factory.getConfigurationName());
     }
 
     public void doStart() throws WaitingException, Exception {
@@ -130,6 +130,13 @@
 
     public void doStop() throws WaitingException, Exception {
         Configuration.setConfiguration(oldConfiguration);
+
+        for (Iterator iter = entries.keySet().iterator(); iter.hasNext();){
+            ConfigurationEntryFactory factory = (ConfigurationEntryFactory) entries.get(iter.next());
+            log.info("Removed Application Configuration Entry " + factory.getConfigurationName());
+        }
+        entries.clear();
+        
         log.info("Uninstalled Geronimo login configuration");
     }
 

Mime
View raw message