You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by tv...@apache.org on 2011/01/21 23:23:38 UTC
svn commit: r1062057 -
/openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/ConfigurationFactory.java
Author: tveronezi
Date: Fri Jan 21 22:23:38 2011
New Revision: 1062057
URL: http://svn.apache.org/viewvc?rev=1062057&view=rev
Log:
this code was throwing the IOException below:
java.io.IOException: The filename, directory name, or volume label syntax is incorrect
Modified:
openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/ConfigurationFactory.java
Modified: openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/ConfigurationFactory.java
URL: http://svn.apache.org/viewvc/openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/ConfigurationFactory.java?rev=1062057&r1=1062056&r2=1062057&view=diff
==============================================================================
--- openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/ConfigurationFactory.java (original)
+++ openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/ConfigurationFactory.java Fri Jan 21 22:23:38 2011
@@ -334,11 +334,20 @@ public class ConfigurationFactory implem
for (String pathname : declaredApps) {
try {
- File jarFile = new File(pathname);
-
- AppInfo appInfo = configureApplication(jarFile);
-
- sys.containerSystem.applications.add(appInfo);
+ try {
+ final File jarFile;
+ if(pathname.startsWith("file:/")) {
+ jarFile = new File(new URI(pathname));
+ } else {
+ jarFile = new File(pathname);
+ }
+
+ AppInfo appInfo = configureApplication(jarFile);
+
+ sys.containerSystem.applications.add(appInfo);
+ } catch (URISyntaxException e) {
+ logger.error("Invalid declaredApp URI '" + pathname + "'", e);
+ }
} catch (OpenEJBException alreadyHandled) {
}
}