Return-Path: Delivered-To: apmail-geronimo-dev-archive@www.apache.org Received: (qmail 44043 invoked from network); 3 Apr 2007 02:50:56 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 3 Apr 2007 02:50:56 -0000 Received: (qmail 13019 invoked by uid 500); 3 Apr 2007 02:51:00 -0000 Delivered-To: apmail-geronimo-dev-archive@geronimo.apache.org Received: (qmail 12954 invoked by uid 500); 3 Apr 2007 02:51:00 -0000 Mailing-List: contact dev-help@geronimo.apache.org; run by ezmlm Precedence: bulk list-help: list-unsubscribe: List-Post: Reply-To: dev@geronimo.apache.org List-Id: Delivered-To: mailing list dev@geronimo.apache.org Received: (qmail 12941 invoked by uid 99); 3 Apr 2007 02:51:00 -0000 Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 02 Apr 2007 19:51:00 -0700 X-ASF-Spam-Status: No, hits=-100.0 required=10.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.4] (HELO brutus.apache.org) (140.211.11.4) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 02 Apr 2007 19:50:52 -0700 Received: from brutus (localhost [127.0.0.1]) by brutus.apache.org (Postfix) with ESMTP id 4AF2F71403D for ; Mon, 2 Apr 2007 19:50:32 -0700 (PDT) Message-ID: <9080712.1175568632271.JavaMail.jira@brutus> Date: Mon, 2 Apr 2007 19:50:32 -0700 (PDT) From: "Don Hill (JIRA)" To: dev@geronimo.apache.org Subject: [jira] Updated: (GERONIMO-2534) Security realms portlet should validate the realm-name for duplicate name In-Reply-To: <20875092.1162214536496.JavaMail.root@brutus> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org [ https://issues.apache.org/jira/browse/GERONIMO-2534?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Don Hill updated GERONIMO-2534: ------------------------------- Attachment: selectType.jsp.patch SecurityRealmPortlet.patch @rev 524947 SecurityRealmPortlet.patch - patch for applications/console/geronimo-console-standard/src/main/java/org/apache/geronimo/console/securitymanager/realm/SecurityRealmPortlet.java added check in processAction which calls new method, sets error message if realm name exists and returns back to select type 2.) selectType.jsp.patch - patch for applications/console/geronimo-console-standard/src/main/webapp/WEB-INF/view/realmwizard/selectType.jsp added message check/display > Security realms portlet should validate the realm-name for duplicate name > ------------------------------------------------------------------------- > > Key: GERONIMO-2534 > URL: https://issues.apache.org/jira/browse/GERONIMO-2534 > Project: Geronimo > Issue Type: Bug > Security Level: public(Regular issues) > Components: console > Affects Versions: 1.1.1, 1.2 > Reporter: Vamsavardhana Reddy > Fix For: 1.x, 2.0-M5 > > Attachments: SecurityRealmPortlet.patch, selectType.jsp.patch > > > "Add new security realm" function does not check if the realm-name entered by the user is already in use. There are no errors shown in the console, whereas the following error is logged: > 18:32:12,033 ERROR [ProxyCollection] Listener threw exception > java.lang.IllegalArgumentException: ConfigurationEntry already registered > at org.apache.geronimo.security.jaas.GeronimoLoginConfiguration.addConfi > guration(GeronimoLoginConfiguration.java:115) > at org.apache.geronimo.security.jaas.GeronimoLoginConfiguration.memberAd > ded(GeronimoLoginConfiguration.java:96) > at org.apache.geronimo.gbean.runtime.ProxyCollection.addTarget(ProxyColl > ection.java:102) > at org.apache.geronimo.gbean.runtime.GBeanCollectionReference.targetAdde > d(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.fireRunningEve > nt(BasicLifecycleMonitor.java:173) > at org.apache.geronimo.kernel.basic.BasicLifecycleMonitor.access$300(Bas > icLifecycleMonitor.java:41) > at org.apache.geronimo.kernel.basic.BasicLifecycleMonitor$RawLifecycleBr > oadcaster.fireRunningEvent(BasicLifecycleMonitor.java:251) > at org.apache.geronimo.gbean.runtime.GBeanInstanceState.attemptFullStart > (GBeanInstanceState.java:292) > at org.apache.geronimo.gbean.runtime.GBeanInstanceState.start(GBeanInsta > nceState.java:102) > at org.apache.geronimo.gbean.runtime.GBeanInstance.start(GBeanInstance.j > ava:526) > at org.apache.geronimo.gbean.runtime.GBeanDependency.attemptFullStart(GB > eanDependency.java:111) > at org.apache.geronimo.gbean.runtime.GBeanDependency.addTarget(GBeanDepe > ndency.java:146) > at org.apache.geronimo.gbean.runtime.GBeanDependency$1.running(GBeanDepe > ndency.java:120) > at org.apache.geronimo.kernel.basic.BasicLifecycleMonitor.fireRunningEve > nt(BasicLifecycleMonitor.java:173) > at org.apache.geronimo.kernel.basic.BasicLifecycleMonitor.access$300(Bas > icLifecycleMonitor.java:41) > at org.apache.geronimo.kernel.basic.BasicLifecycleMonitor$RawLifecycleBr > oadcaster.fireRunningEvent(BasicLifecycleMonitor.java:251) > at org.apache.geronimo.gbean.runtime.GBeanInstanceState.attemptFullStart > (GBeanInstanceState.java:292) > at org.apache.geronimo.gbean.runtime.GBeanInstanceState.start(GBeanInsta > nceState.java:102) > at org.apache.geronimo.gbean.runtime.GBeanInstance.start(GBeanInstance.j > ava:526) > at org.apache.geronimo.gbean.runtime.GBeanDependency.attemptFullStart(GB > eanDependency.java:111) > at org.apache.geronimo.gbean.runtime.GBeanDependency.addTarget(GBeanDepe > ndency.java:146) > at org.apache.geronimo.gbean.runtime.GBeanDependency$1.running(GBeanDepe > ndency.java:120) > at org.apache.geronimo.kernel.basic.BasicLifecycleMonitor.fireRunningEve > nt(BasicLifecycleMonitor.java:173) > at org.apache.geronimo.kernel.basic.BasicLifecycleMonitor.access$300(Bas > icLifecycleMonitor.java:41) > at org.apache.geronimo.kernel.basic.BasicLifecycleMonitor$RawLifecycleBr > oadcaster.fireRunningEvent(BasicLifecycleMonitor.java:251) > at org.apache.geronimo.gbean.runtime.GBeanInstanceState.attemptFullStart > (GBeanInstanceState.java:292) > at org.apache.geronimo.gbean.runtime.GBeanInstanceState.start(GBeanInsta > nceState.java:102) > at org.apache.geronimo.gbean.runtime.GBeanInstanceState.startRecursive(G > BeanInstanceState.java:124) > at org.apache.geronimo.gbean.runtime.GBeanInstance.startRecursive(GBeanI > nstance.java:540) > at org.apache.geronimo.kernel.basic.BasicKernel.startRecursiveGBean(Basi > cKernel.java:379) > at org.apache.geronimo.kernel.config.ConfigurationUtil.startConfiguratio > nGBeans(ConfigurationUtil.java:374) > at org.apache.geronimo.kernel.config.KernelConfigurationManager.start(Ke > rnelConfigurationManager.java:187) > at org.apache.geronimo.kernel.config.SimpleConfigurationManager.startCon > figuration(SimpleConfigurationManager.java:527) > at org.apache.geronimo.kernel.config.SimpleConfigurationManager.startCon > figuration(SimpleConfigurationManager.java:508) > at org.apache.geronimo.kernel.config.SimpleConfigurationManager$$FastCla > ssByCGLIB$$ce77a924.invoke() > at net.sf.cglib.reflect.FastMethod.invoke(FastMethod.java:53) > at org.apache.geronimo.gbean.runtime.FastMethodInvoker.invoke(FastMethod > Invoker.java:38) > at org.apache.geronimo.gbean.runtime.GBeanOperation.invoke(GBeanOperatio > n.java:122) > at org.apache.geronimo.gbean.runtime.GBeanInstance.invoke(GBeanInstance. > java:817) > at org.apache.geronimo.gbean.runtime.RawInvoker.invoke(RawInvoker.java:5 > 7) > at org.apache.geronimo.kernel.basic.RawOperationInvoker.invoke(RawOperat > ionInvoker.java:35) > at org.apache.geronimo.kernel.basic.ProxyMethodInterceptor.intercept(Pro > xyMethodInterceptor.java:96) > at org.apache.geronimo.kernel.config.EditableConfigurationManager$$Enhan > cerByCGLIB$$cb9efa2b.startConfiguration() > at org.apache.geronimo.deployment.plugin.local.StartCommand.run(StartCom > mand.java:67) > at java.lang.Thread.run(Unknown Source) -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.