[ 
http://issues.apache.org/jira/browse/GERONIMO-1150?page=comments#action_12357194
 ] 

David Jencks commented on GERONIMO-1150:
----------------------------------------

Schema validation can't fully validate the security info.  If you specify a 
security-realm-name to allow login, you should also specify a security 
configuration.  However this can be specified either in the web plan or in an 
enclosing ear plan or in any other module in the ear.

My preference would be that specifying a security-realm-name with no security 
result in a DeploymentException.  However if there is strong opposition I'd be 
willing to settle for a warning.  I've implemented the exception pending 
further discussion.

Sending        
modules/jetty-builder/src/java/org/apache/geronimo/jetty/deployment/JettyModuleBuilder.java
Sending        
modules/tomcat-builder/src/java/org/apache/geronimo/tomcat/deployment/TomcatModuleBuilder.java
Transmitting file data ..
Committed revision 332225.

> NullPointerException when no security element is in plan
> --------------------------------------------------------
>
>          Key: GERONIMO-1150
>          URL: http://issues.apache.org/jira/browse/GERONIMO-1150
>      Project: Geronimo
>         Type: Bug
>   Components: web, deployment
>     Versions: 1.0-M5
>     Reporter: Jacek Laskowski
>     Assignee: Jacek Laskowski
>      Fix For: 1.0

>
> When no security element is specified in a plan, the following 
> NullPointerException is thrown.
>     [java] Caused by: java.lang.NullPointerException
>     [java]      at 
> org.apache.geronimo.jetty.deployment.JettyModuleBuilder.addGBeans(JettyModuleBuilder.java:815)
>     [java]      ... 43 more
> According to the xsd - jetty-builder\src\schema\geronimo-jetty-1.0.xsd - and 
> JettyModuleBuilder the element is not mandatory.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira

Reply via email to