Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 318894 - JBoss-jetty fails to start when jboss home is directory with spaces in it
Summary: JBoss-jetty fails to start when jboss home is directory with spaces in it
Status: RESOLVED FIXED
Alias: None
Product: Jetty
Classification: RT
Component: other (show other bugs)
Version: 7.1.3   Edit
Hardware: All Windows All
: P3 normal (vote)
Target Milestone: 7.1.x   Edit
Assignee: Greg Wilkins CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-05 09:42 EDT by Jeremy Stone CLA
Modified: 2010-07-06 12:42 EDT (History)
1 user (show)

See Also:

Attachments
Stack trace (10.53 KB, text/plain)
2010-07-05 09:42 EDT, Jeremy Stone CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jeremy Stone CLA 2010-07-05 09:42:38 EDT 
Created attachment 173429 [details]
Stack trace

There is an issue in JBoss (JBoss AS 4.3.0.GA.CP04) whereby if the installation directory name has spaces in it the URLs used internally are not escaped properly. 

The result is that invalid deployment URLs are passed to the org.jboss.jetty.JettyDeployer.performDeploy(...) method which responds by throwing a URISyntaxException.

This seems different from Bug 285906 which looks to be related to standalone Jetty deployment.

I realize that this is really a bug in JBoss but JBoss works fine with its bundled Tomcat so I'm not sure they will be too interested in fixing it. The bundled Tomcat is no good for us as we need Jetty's Websockets support.

To fix our particular issue with spaces we patched JettyDeployer to add:
 warUrl = warUrl.replace(" ", "%20");
at the top of the performDeploy and performUndeploy methods.
Comment 1 Michael Gorovoy CLA 2010-07-06 12:42:22 EDT 
Committed r6093 by Greg W.