From 14eab131217f45b1c508227eb5fbc301697f6a8b Mon Sep 17 00:00:00 2001 From: Arjen Poutsma Date: Sun, 17 Feb 2008 02:27:48 +0000 Subject: [PATCH] Added HTTP method constants. --- .../ws/transport/http/HttpTransportConstants.java | 6 ++++++ .../http/HttpUrlConnectionMessageSender.java | 4 +--- .../transport/http/MessageDispatcherServlet.java | 3 ++- .../WebServiceMessageReceiverHandlerAdapter.java | 2 +- .../http/WsdlDefinitionHandlerAdapter.java | 7 ++++--- .../http/MessageDispatcherServletTest.java | 6 ++++-- ...bServiceMessageReceiverHandlerAdapterTest.java | 11 ++++++----- .../http/WsdlDefinitionHandlerAdapterTest.java | 15 ++++++++------- 8 files changed, 32 insertions(+), 22 deletions(-) diff --git a/core/src/main/java/org/springframework/ws/transport/http/HttpTransportConstants.java b/core/src/main/java/org/springframework/ws/transport/http/HttpTransportConstants.java index 28376cac..0ed01433 100644 --- a/core/src/main/java/org/springframework/ws/transport/http/HttpTransportConstants.java +++ b/core/src/main/java/org/springframework/ws/transport/http/HttpTransportConstants.java @@ -55,4 +55,10 @@ public interface HttpTransportConstants extends TransportConstants { /** The "https" URI scheme. */ String HTTPS_URI_SCHEME = "https"; + + /** The "GET" HTTP method */ + String METHOD_GET = "GET"; + + /** The "POST" HTTP method */ + String METHOD_POST = "POST"; } diff --git a/core/src/main/java/org/springframework/ws/transport/http/HttpUrlConnectionMessageSender.java b/core/src/main/java/org/springframework/ws/transport/http/HttpUrlConnectionMessageSender.java index f91ad4f4..19dd357b 100644 --- a/core/src/main/java/org/springframework/ws/transport/http/HttpUrlConnectionMessageSender.java +++ b/core/src/main/java/org/springframework/ws/transport/http/HttpUrlConnectionMessageSender.java @@ -38,8 +38,6 @@ import org.springframework.ws.transport.WebServiceConnection; */ public class HttpUrlConnectionMessageSender extends AbstractHttpWebServiceMessageSender { - private static final String HTTP_METHOD_POST = "POST"; - public WebServiceConnection createConnection(URI uri) throws IOException { URL url = uri.toURL(); URLConnection connection = url.openConnection(); @@ -48,7 +46,7 @@ public class HttpUrlConnectionMessageSender extends AbstractHttpWebServiceMessag } else { HttpURLConnection httpURLConnection = (HttpURLConnection) connection; - httpURLConnection.setRequestMethod(HTTP_METHOD_POST); + httpURLConnection.setRequestMethod(HttpTransportConstants.METHOD_POST); httpURLConnection.setUseCaches(false); httpURLConnection.setDoInput(true); httpURLConnection.setDoOutput(true); diff --git a/core/src/main/java/org/springframework/ws/transport/http/MessageDispatcherServlet.java b/core/src/main/java/org/springframework/ws/transport/http/MessageDispatcherServlet.java index 8c7bd8a4..50f4ca8d 100644 --- a/core/src/main/java/org/springframework/ws/transport/http/MessageDispatcherServlet.java +++ b/core/src/main/java/org/springframework/ws/transport/http/MessageDispatcherServlet.java @@ -231,7 +231,8 @@ public class MessageDispatcherServlet extends FrameworkServlet { * @return a definition, or null */ protected WsdlDefinition getWsdlDefinition(HttpServletRequest request) { - if ("GET".equals(request.getMethod()) && request.getRequestURI().endsWith(WSDL_SUFFIX_NAME)) { + if (HttpTransportConstants.METHOD_GET.equals(request.getMethod()) && + request.getRequestURI().endsWith(WSDL_SUFFIX_NAME)) { String fileName = WebUtils.extractFilenameFromUrlPath(request.getRequestURI()); return (WsdlDefinition) wsdlDefinitions.get(fileName); } diff --git a/core/src/main/java/org/springframework/ws/transport/http/WebServiceMessageReceiverHandlerAdapter.java b/core/src/main/java/org/springframework/ws/transport/http/WebServiceMessageReceiverHandlerAdapter.java index 9c420296..0c0dc7f3 100644 --- a/core/src/main/java/org/springframework/ws/transport/http/WebServiceMessageReceiverHandlerAdapter.java +++ b/core/src/main/java/org/springframework/ws/transport/http/WebServiceMessageReceiverHandlerAdapter.java @@ -52,7 +52,7 @@ public class WebServiceMessageReceiverHandlerAdapter extends WebServiceMessageRe public ModelAndView handle(HttpServletRequest httpServletRequest, HttpServletResponse httpServletResponse, Object handler) throws Exception { - if ("POST".equals(httpServletRequest.getMethod())) { + if (HttpTransportConstants.METHOD_POST.equals(httpServletRequest.getMethod())) { WebServiceConnection connection = new HttpServletConnection(httpServletRequest, httpServletResponse); handleConnection(connection, (WebServiceMessageReceiver) handler); } diff --git a/core/src/main/java/org/springframework/ws/transport/http/WsdlDefinitionHandlerAdapter.java b/core/src/main/java/org/springframework/ws/transport/http/WsdlDefinitionHandlerAdapter.java index 48eda361..3ae8f482 100644 --- a/core/src/main/java/org/springframework/ws/transport/http/WsdlDefinitionHandlerAdapter.java +++ b/core/src/main/java/org/springframework/ws/transport/http/WsdlDefinitionHandlerAdapter.java @@ -29,6 +29,9 @@ import javax.xml.transform.stream.StreamResult; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; +import org.w3c.dom.Attr; +import org.w3c.dom.Document; + import org.springframework.beans.factory.InitializingBean; import org.springframework.util.StringUtils; import org.springframework.web.servlet.HandlerAdapter; @@ -37,8 +40,6 @@ import org.springframework.ws.wsdl.WsdlDefinition; import org.springframework.xml.transform.TransformerObjectSupport; import org.springframework.xml.xpath.XPathExpression; import org.springframework.xml.xpath.XPathExpressionFactory; -import org.w3c.dom.Attr; -import org.w3c.dom.Document; /** * Adapter to use the WsdlDefinition interface with the generic DispatcherServlet. @@ -113,7 +114,7 @@ public class WsdlDefinitionHandlerAdapter extends TransformerObjectSupport imple public ModelAndView handle(HttpServletRequest request, HttpServletResponse response, Object handler) throws Exception { - if ("GET".equals(request.getMethod())) { + if (HttpTransportConstants.METHOD_GET.equals(request.getMethod())) { response.setContentType(CONTENT_TYPE); Transformer transformer = createTransformer(); WsdlDefinition definition = (WsdlDefinition) handler; diff --git a/core/src/test/java/org/springframework/ws/transport/http/MessageDispatcherServletTest.java b/core/src/test/java/org/springframework/ws/transport/http/MessageDispatcherServletTest.java index f654dffc..bfdcf9c3 100644 --- a/core/src/test/java/org/springframework/ws/transport/http/MessageDispatcherServletTest.java +++ b/core/src/test/java/org/springframework/ws/transport/http/MessageDispatcherServletTest.java @@ -9,6 +9,8 @@ import javax.xml.parsers.DocumentBuilderFactory; import org.custommonkey.xmlunit.XMLTestCase; import org.custommonkey.xmlunit.XMLUnit; +import org.w3c.dom.Document; + import org.springframework.beans.BeansException; import org.springframework.beans.MutablePropertyValues; import org.springframework.core.io.ClassPathResource; @@ -22,7 +24,6 @@ import org.springframework.ws.server.endpoint.adapter.PayloadEndpointAdapter; import org.springframework.ws.server.endpoint.mapping.PayloadRootQNameEndpointMapping; import org.springframework.ws.soap.server.endpoint.SimpleSoapExceptionResolver; import org.springframework.ws.wsdl.wsdl11.SimpleWsdl11Definition; -import org.w3c.dom.Document; public class MessageDispatcherServletTest extends XMLTestCase { @@ -61,7 +62,8 @@ public class MessageDispatcherServletTest extends XMLTestCase { public void testDetectWsdlDefinitions() throws Exception { servlet.setContextClass(WsdlDefinitionWebApplicationContext.class); servlet.init(config); - MockHttpServletRequest request = new MockHttpServletRequest("GET", "/definition.wsdl"); + MockHttpServletRequest request = + new MockHttpServletRequest(HttpTransportConstants.METHOD_GET, "/definition.wsdl"); MockHttpServletResponse response = new MockHttpServletResponse(); servlet.service(request, response); DocumentBuilderFactory documentBuilderFactory = DocumentBuilderFactory.newInstance(); diff --git a/core/src/test/java/org/springframework/ws/transport/http/WebServiceMessageReceiverHandlerAdapterTest.java b/core/src/test/java/org/springframework/ws/transport/http/WebServiceMessageReceiverHandlerAdapterTest.java index 185c2c38..906d1ea9 100644 --- a/core/src/test/java/org/springframework/ws/transport/http/WebServiceMessageReceiverHandlerAdapterTest.java +++ b/core/src/test/java/org/springframework/ws/transport/http/WebServiceMessageReceiverHandlerAdapterTest.java @@ -20,6 +20,7 @@ import javax.servlet.http.HttpServletResponse; import junit.framework.TestCase; import org.easymock.MockControl; + import org.springframework.mock.web.MockHttpServletRequest; import org.springframework.mock.web.MockHttpServletResponse; import org.springframework.ws.FaultAwareWebServiceMessage; @@ -65,7 +66,7 @@ public class WebServiceMessageReceiverHandlerAdapterTest extends TestCase { } public void testHandleNonPost() throws Exception { - httpRequest.setMethod("GET"); + httpRequest.setMethod(HttpTransportConstants.METHOD_GET); replayMockControls(); WebServiceMessageReceiver endpoint = new WebServiceMessageReceiver() { @@ -79,7 +80,7 @@ public class WebServiceMessageReceiverHandlerAdapterTest extends TestCase { } public void testHandlePostNoResponse() throws Exception { - httpRequest.setMethod("POST"); + httpRequest.setMethod(HttpTransportConstants.METHOD_POST); httpRequest.setContent(REQUEST.getBytes("UTF-8")); httpRequest.setContentType("text/xml; charset=\"utf-8\""); httpRequest.setCharacterEncoding("UTF-8"); @@ -102,7 +103,7 @@ public class WebServiceMessageReceiverHandlerAdapterTest extends TestCase { } public void testHandlePostResponse() throws Exception { - httpRequest.setMethod("POST"); + httpRequest.setMethod(HttpTransportConstants.METHOD_POST); httpRequest.setContent(REQUEST.getBytes("UTF-8")); httpRequest.setContentType("text/xml; charset=\"utf-8\""); httpRequest.setCharacterEncoding("UTF-8"); @@ -129,7 +130,7 @@ public class WebServiceMessageReceiverHandlerAdapterTest extends TestCase { } public void testHandlePostFault() throws Exception { - httpRequest.setMethod("POST"); + httpRequest.setMethod(HttpTransportConstants.METHOD_POST); httpRequest.setContent(REQUEST.getBytes("UTF-8")); httpRequest.setContentType("text/xml; charset=\"utf-8\""); httpRequest.setCharacterEncoding("UTF-8"); @@ -157,7 +158,7 @@ public class WebServiceMessageReceiverHandlerAdapterTest extends TestCase { } public void testHandleNotFound() throws Exception { - httpRequest.setMethod("POST"); + httpRequest.setMethod(HttpTransportConstants.METHOD_POST); httpRequest.setContent(REQUEST.getBytes("UTF-8")); httpRequest.setContentType("text/xml; charset=\"utf-8\""); httpRequest.setCharacterEncoding("UTF-8"); diff --git a/core/src/test/java/org/springframework/ws/transport/http/WsdlDefinitionHandlerAdapterTest.java b/core/src/test/java/org/springframework/ws/transport/http/WsdlDefinitionHandlerAdapterTest.java index 0fa65ddc..529b160a 100644 --- a/core/src/test/java/org/springframework/ws/transport/http/WsdlDefinitionHandlerAdapterTest.java +++ b/core/src/test/java/org/springframework/ws/transport/http/WsdlDefinitionHandlerAdapterTest.java @@ -28,14 +28,15 @@ import javax.xml.parsers.DocumentBuilderFactory; import org.custommonkey.xmlunit.XMLTestCase; import org.easymock.MockControl; +import org.w3c.dom.Document; +import org.xml.sax.InputSource; + import org.springframework.core.io.ClassPathResource; import org.springframework.mock.web.MockHttpServletRequest; import org.springframework.mock.web.MockHttpServletResponse; import org.springframework.ws.wsdl.WsdlDefinition; import org.springframework.ws.wsdl.wsdl11.SimpleWsdl11Definition; import org.springframework.xml.transform.StringSource; -import org.w3c.dom.Document; -import org.xml.sax.InputSource; public class WsdlDefinitionHandlerAdapterTest extends XMLTestCase { @@ -59,7 +60,7 @@ public class WsdlDefinitionHandlerAdapterTest extends XMLTestCase { } public void testHandleGet() throws Exception { - request.setMethod("GET"); + request.setMethod(HttpTransportConstants.METHOD_GET); String definition = ""; definitionControl.expectAndReturn(definitionMock.getSource(), new StringSource(definition)); definitionControl.replay(); @@ -69,7 +70,7 @@ public class WsdlDefinitionHandlerAdapterTest extends XMLTestCase { } public void testHandleNonGet() throws Exception { - request.setMethod("POST"); + request.setMethod(HttpTransportConstants.METHOD_POST); definitionControl.replay(); adapter.handle(request, response, definitionMock); definitionControl.verify(); @@ -79,7 +80,7 @@ public class WsdlDefinitionHandlerAdapterTest extends XMLTestCase { public void testTransformLocations() throws Exception { adapter.setTransformLocations(true); - request.setMethod("GET"); + request.setMethod(HttpTransportConstants.METHOD_GET); request.setScheme("http"); request.setServerName("example.com"); request.setServerPort(8080); @@ -156,7 +157,7 @@ public class WsdlDefinitionHandlerAdapterTest extends XMLTestCase { public void testHandleSimpleWsdl11DefinitionWithoutTransformLocations() throws Exception { adapter.setTransformLocations(false); - request.setMethod("GET"); + request.setMethod(HttpTransportConstants.METHOD_GET); request.setScheme("http"); request.setServerName("example.com"); request.setServerPort(8080); @@ -189,7 +190,7 @@ public class WsdlDefinitionHandlerAdapterTest extends XMLTestCase { public void testHandleSimpleWsdl11DefinitionWithTransformLocation() throws Exception { adapter.setTransformLocations(true); - request.setMethod("GET"); + request.setMethod(HttpTransportConstants.METHOD_GET); request.setScheme("http"); request.setServerName("example.com"); request.setServerPort(8080);