Details
-
Bug
-
Status: Resolved
-
Minor
-
Resolution: Fixed
-
servicemix-http-2009.01
-
None
Description
I'm in the process of adding unit tests for the servicemix-http component. One of the classes in this component is called ManagedContextManager. It has a method called createContext() to add a context for a given URL and HTTP processor to the map of managed contexts. There is some code in that method that is confusing.
Following is the implementation of the method:
public synchronized Object createContext(String strUrl, HttpProcessor processor) throws Exception {
URI uri = new URI(strUrl);
String path = uri.getPath();
if (!path.startsWith("/"))
if (!path.endsWith("/")) { path = path + "/"; }
managedContexts.put(path, processor);
return path;
}
It seems like if the path does not begin with "/" that it should be PRE-pended to the path (not appended). I will attach a patch to fix this problem.