[jira] Issue Comment Edited: (GERONIMO-4553) Admin console does not show error when creating duplicate security realm

2009-07-28 Thread Rex Wang (JIRA)

[ 
https://issues.apache.org/jira/browse/GERONIMO-4553?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12735977#action_12735977
 ] 

Rex Wang edited comment on GERONIMO-4553 at 7/28/09 5:28 AM:
-

Sorry, I had a typo in ahead comments... I should said Server-wide not 
Server-side...:-)
Anyway, I used some time to understand the difference between 
server-wide/non-server-wide and global/non-global, IIUC, the server-wide means 
the realm is a standalone realm.  If we want duplicate-named security realms, 
they can not  be both global. Right?

Thanks
-Rex

  was (Author: rwonly):
Sorry, I had a typo in ahead comments... I should said Server-wide not 
Server-side...:-)
Anyway, I used some time to understand the difference between 
server-wide/non-server-wide and global/non-global, IIUC, the server-wide means 
the realm is a standalone realm. And both global and no-global realm are must 
be a server-wide realm. If we want duplicate-named security realms, they can 
not  be both global. Right?

Thanks
-Rex
  
 Admin console does not show error when creating duplicate security realm
 

 Key: GERONIMO-4553
 URL: https://issues.apache.org/jira/browse/GERONIMO-4553
 Project: Geronimo
  Issue Type: Bug
  Security Level: public(Regular issues) 
  Components: console, security
Affects Versions: 2.1.4, 2.2
Reporter: David Jencks
Assignee: Rex Wang
 Fix For: 2.1.5, 2.2

 Attachments: GERONIMO-4553-b21-updated.patch, GERONIMO-4553-b21.patch


 If you create a security realm with a duplicate name (such as geronimo-admin) 
 using the admin console, everything appears to work in the ui however the 
 command line console shows the error:
 2009-02-24 09:47:11,123 ERROR [ProxyCollection] Listener threw exception
 java.lang.IllegalArgumentException: ConfigurationEntry named: geronimo-admin 
 already registered
 at 
 org.apache.geronimo.security.jaas.GeronimoLoginConfiguration.addConfiguration(GeronimoLoginConfiguration.java:112)
 at 
 org.apache.geronimo.security.jaas.GeronimoLoginConfiguration.memberAdded(GeronimoLoginConfiguration.java:97)
 at 
 org.apache.geronimo.gbean.runtime.ProxyCollection.addTarget(ProxyCollection.java:102)
 at 
 org.apache.geronimo.gbean.runtime.GBeanCollectionReference.targetAdded(GBeanCollectionReference.java:96)
 at 
 org.apache.geronimo.gbean.runtime.GBeanCollectionReference.addTarget(GBeanCollectionReference.java:180)
 at 
 org.apache.geronimo.gbean.runtime.GBeanCollectionReference$1.running(GBeanCollectionReference.java:110)
 at 
 org.apache.geronimo.kernel.basic.BasicLifecycleMonitor.fireRunningEvent(BasicLifecycleMonitor.java:175)
 at 
 org.apache.geronimo.kernel.basic.BasicLifecycleMonitor.access$300(BasicLifecycleMonitor.java:44)
 at 
 org.apache.geronimo.kernel.basic.BasicLifecycleMonitor$RawLifecycleBroadcaster.fireRunningEvent(BasicLifecycleMonitor.java:253)
 at 
 org.apache.geronimo.gbean.runtime.GBeanInstanceState.attemptFullStart(GBeanInstanceState.java:295)
 at 
 org.apache.geronimo.gbean.runtime.GBeanInstanceState.start(GBeanInstanceState.java:103)
 at 
 org.apache.geronimo.gbean.runtime.GBeanInstance.start(GBeanInstance.java:524)
 at 
 org.apache.geronimo.gbean.runtime.GBeanDependency.attemptFullStart(GBeanDependency.java:110)
 at 
 org.apache.geronimo.gbean.runtime.GBeanDependency.addTarget(GBeanDependency.java:145)
 at 
 org.apache.geronimo.gbean.runtime.GBeanDependency$1.running(GBeanDependency.java:119)
 at 
 org.apache.geronimo.kernel.basic.BasicLifecycleMonitor.fireRunningEvent(BasicLifecycleMonitor.java:175)
 at 
 org.apache.geronimo.kernel.basic.BasicLifecycleMonitor.access$300(BasicLifecycleMonitor.java:44)
 at 
 org.apache.geronimo.kernel.basic.BasicLifecycleMonitor$RawLifecycleBroadcaster.fireRunningEvent(BasicLifecycleMonitor.java:253)
 at 
 org.apache.geronimo.gbean.runtime.GBeanInstanceState.attemptFullStart(GBeanInstanceState.java:295)
 at 
 org.apache.geronimo.gbean.runtime.GBeanInstanceState.start(GBeanInstanceState.java:103)
 at 
 org.apache.geronimo.gbean.runtime.GBeanInstance.start(GBeanInstance.java:524)
 at 
 org.apache.geronimo.gbean.runtime.GBeanDependency.attemptFullStart(GBeanDependency.java:110)
 at 
 org.apache.geronimo.gbean.runtime.GBeanDependency.addTarget(GBeanDependency.java:145)
 at 
 org.apache.geronimo.gbean.runtime.GBeanDependency$1.running(GBeanDependency.java:119)
 at 
 org.apache.geronimo.kernel.basic.BasicLifecycleMonitor.fireRunningEvent(BasicLifecycleMonitor.java:175)
 at 
 

[jira] Issue Comment Edited: (GERONIMO-4553) Admin console does not show error when creating duplicate security realm

2009-03-03 Thread Joe Bohn (JIRA)

[ 
https://issues.apache.org/jira/browse/GERONIMO-4553?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12678457#action_12678457
 ] 

jbohn edited comment on GERONIMO-4553 at 3/3/09 2:29 PM:


Hmmm ... things are a bit more strange than I thought.  I can create a realm 
named geronimo-admin (no error is returned to the portlet at all).  The name is 
different than our default realm (which is geronimo-realm) - but there is still 
the error that you mentioned originally.   If I attempt to create a second 
geronimo-admin realm then I get a different error (that is echoed back to the 
portlet) and my code will display an error to the user.   So there is more 
going on here than I first thought.   I think the change I made is still valid 
but I'm not sure it addresses  this issue completely.  Thoughts? 

  was (Author: jbohn):
Hmmm ... things are a bit more strange than I thought.  I can realm named 
geronimo-admin (no error is returned to the portlet at all).  The name is 
different than our default realm (which is geronimo-realm) - but there is still 
the error that you mentioned originally.   If I attempt to create a second 
geronimo-admin realm then I get a different error (that is echoed back to the 
portlet) and my code will display an error.   So there is more going on here 
than I first thought.   I think the change I made is still valid but I'm not 
sure it addresses  this issue completely.  Thoughts? 
  
 Admin console does not show error when creating duplicate security realm
 

 Key: GERONIMO-4553
 URL: https://issues.apache.org/jira/browse/GERONIMO-4553
 Project: Geronimo
  Issue Type: Bug
  Security Level: public(Regular issues) 
  Components: console, security
Affects Versions: 2.1.4, 2.2
Reporter: David Jencks
Assignee: Joe Bohn
 Fix For: 2.1.4, 2.2


 If you create a security realm with a duplicate name (such as geronimo-admin) 
 using the admin console, everything appears to work in the ui however the 
 command line console shows the error:
 2009-02-24 09:47:11,123 ERROR [ProxyCollection] Listener threw exception
 java.lang.IllegalArgumentException: ConfigurationEntry named: geronimo-admin 
 already registered
 at 
 org.apache.geronimo.security.jaas.GeronimoLoginConfiguration.addConfiguration(GeronimoLoginConfiguration.java:112)
 at 
 org.apache.geronimo.security.jaas.GeronimoLoginConfiguration.memberAdded(GeronimoLoginConfiguration.java:97)
 at 
 org.apache.geronimo.gbean.runtime.ProxyCollection.addTarget(ProxyCollection.java:102)
 at 
 org.apache.geronimo.gbean.runtime.GBeanCollectionReference.targetAdded(GBeanCollectionReference.java:96)
 at 
 org.apache.geronimo.gbean.runtime.GBeanCollectionReference.addTarget(GBeanCollectionReference.java:180)
 at 
 org.apache.geronimo.gbean.runtime.GBeanCollectionReference$1.running(GBeanCollectionReference.java:110)
 at 
 org.apache.geronimo.kernel.basic.BasicLifecycleMonitor.fireRunningEvent(BasicLifecycleMonitor.java:175)
 at 
 org.apache.geronimo.kernel.basic.BasicLifecycleMonitor.access$300(BasicLifecycleMonitor.java:44)
 at 
 org.apache.geronimo.kernel.basic.BasicLifecycleMonitor$RawLifecycleBroadcaster.fireRunningEvent(BasicLifecycleMonitor.java:253)
 at 
 org.apache.geronimo.gbean.runtime.GBeanInstanceState.attemptFullStart(GBeanInstanceState.java:295)
 at 
 org.apache.geronimo.gbean.runtime.GBeanInstanceState.start(GBeanInstanceState.java:103)
 at 
 org.apache.geronimo.gbean.runtime.GBeanInstance.start(GBeanInstance.java:524)
 at 
 org.apache.geronimo.gbean.runtime.GBeanDependency.attemptFullStart(GBeanDependency.java:110)
 at 
 org.apache.geronimo.gbean.runtime.GBeanDependency.addTarget(GBeanDependency.java:145)
 at 
 org.apache.geronimo.gbean.runtime.GBeanDependency$1.running(GBeanDependency.java:119)
 at 
 org.apache.geronimo.kernel.basic.BasicLifecycleMonitor.fireRunningEvent(BasicLifecycleMonitor.java:175)
 at 
 org.apache.geronimo.kernel.basic.BasicLifecycleMonitor.access$300(BasicLifecycleMonitor.java:44)
 at 
 org.apache.geronimo.kernel.basic.BasicLifecycleMonitor$RawLifecycleBroadcaster.fireRunningEvent(BasicLifecycleMonitor.java:253)
 at 
 org.apache.geronimo.gbean.runtime.GBeanInstanceState.attemptFullStart(GBeanInstanceState.java:295)
 at 
 org.apache.geronimo.gbean.runtime.GBeanInstanceState.start(GBeanInstanceState.java:103)
 at 
 org.apache.geronimo.gbean.runtime.GBeanInstance.start(GBeanInstance.java:524)
 at 
 org.apache.geronimo.gbean.runtime.GBeanDependency.attemptFullStart(GBeanDependency.java:110)
 at