CXF-5610 clean up the code
Project: http://git-wip-us.apache.org/repos/asf/cxf/repo Commit: http://git-wip-us.apache.org/repos/asf/cxf/commit/7f7be98e Tree: http://git-wip-us.apache.org/repos/asf/cxf/tree/7f7be98e Diff: http://git-wip-us.apache.org/repos/asf/cxf/diff/7f7be98e Branch: refs/heads/master Commit: 7f7be98e97e15015bc5303caa4db66615c0b4e81 Parents: c542198 Author: Willem Jiang <[email protected]> Authored: Fri Apr 4 15:20:45 2014 +0800 Committer: Willem Jiang <[email protected]> Committed: Fri Apr 4 15:53:16 2014 +0800 ---------------------------------------------------------------------- .../transport/http_jetty/JettyHTTPServerEngine.java | 9 +-------- .../http_jetty/JettyHTTPServerEngineTest.java | 15 --------------- 2 files changed, 1 insertion(+), 23 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/cxf/blob/7f7be98e/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngine.java ---------------------------------------------------------------------- diff --git a/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngine.java b/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngine.java index 8c40149..073805b 100644 --- a/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngine.java +++ b/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngine.java @@ -277,16 +277,9 @@ public class JettyHTTPServerEngine protected void checkRegistedContext(URL url) { String path = url.getPath(); for (String registedPath : registedPaths) { - if (path.equals(registedPath)) { - //|| HttpUrlUtil.checkContextPath(registedPath, path)) { - // Throw the address is already used exception + if (path.equals(registedPath)) { throw new Fault(new Message("ADD_HANDLER_CONTEXT_IS_USED_MSG", LOG, url, registedPath)); } - /* - if (HttpUrlUtil.checkContextPath(path, registedPath)) { - throw new Fault(new Message("ADD_HANDLER_CONTEXT_CONFILICT_MSG", LOG, url, registedPath)); - } - */ } } http://git-wip-us.apache.org/repos/asf/cxf/blob/7f7be98e/rt/transports/http-jetty/src/test/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngineTest.java ---------------------------------------------------------------------- diff --git a/rt/transports/http-jetty/src/test/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngineTest.java b/rt/transports/http-jetty/src/test/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngineTest.java index 5132379..94aa022 100644 --- a/rt/transports/http-jetty/src/test/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngineTest.java +++ b/rt/transports/http-jetty/src/test/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngineTest.java @@ -199,21 +199,6 @@ public class JettyHTTPServerEngineTest extends Assert { } catch (Exception ex) { assertTrue("Get a wrong exception message", ex.getMessage().indexOf("hello/test") > 0); } - /* - try { - engine.addServant(new URL(urlStr + "/test"), handler2); - fail("We don't support to publish the two service at the same context path"); - } catch (Exception ex) { - assertTrue("Get a wrong exception message", ex.getMessage().indexOf("hello/test/test") > 0); - } - - try { - engine.addServant(new URL("http://localhost:"; + PORT1 + "/hello"), handler2); - fail("We don't support to publish the two service at the same context path"); - } catch (Exception ex) { - assertTrue("Get a wrong exception message", ex.getMessage().indexOf("hello") > 0); - } - */ engine.addServant(new URL(urlStr2), handler2);
