o.a.g.system.configuration.LocalAttributeManager.load() does not setErrorHandler when parsing ----------------------------------------------------------------------------------------------
Key: GERONIMO-2069 URL: http://issues.apache.org/jira/browse/GERONIMO-2069 Project: Geronimo Type: Bug Security: public (Regular issues) Components: startup/shutdown Versions: 1.1 Reporter: John Sisson Assigned to: John Sisson Priority: Minor Fix For: 1.1 There are two issues in the load() processing: * If a validation error occurs, the file containing the error isn't identified. The code needs to use inputsource.setSystemId(..) so the errors from the parser identify the config.xml file, rather than displaying URI=null. * No error handler is set up, so a warning message is issued from Xerces. If the config.xml file is invalid, startup will produce output similar to the following: $./geronimo.sh run --long Using GERONIMO_BASE: /home/sissonj/OpenSourceJava/asf/geronimo/branches/1.1/assemblies/j2ee-tomcat-server/target/geronimo-1.1-SNAPSHOT Using GERONIMO_HOME: /home/sissonj/OpenSourceJava/asf/geronimo/branches/1.1/assemblies/j2ee-tomcat-server/target/geronimo-1.1-SNAPSHOT Using GERONIMO_TMPDIR: /home/sissonj/OpenSourceJava/asf/geronimo/branches/1.1/assemblies/j2ee-tomcat-server/target/geronimo-1.1-SNAPSHOT/var/temp Using JRE_HOME: /usr/j2se Booting Geronimo Kernel (in Java 1.4.2_10)... Warning: validation was turned on but an org.xml.sax.ErrorHandler was not set, which is probably not what is desired. Parser will use a default ErrorHandler to print the first 10 errors. Please call the 'setErrorHandler' method to fix this. Error: URI=null Line=74: cvc-complex-type.2.4.b: The content of element 'reference' is not complete. One of '{"http://geronimo.apache.org/xml/ns/attributes-1.1":pattern}' is expected. Module 1/20 geronimo/rmi-naming/1.1-SNAPSHOT/car started in .594s <SNIP> -- 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