[jbossws-commits] JBossWS SVN: r10901 - stack/native/branches/ropalka/modules/core/src/main/java/org/jboss/ws/core/jaxws/spi/http.

[jbossws-commits at lists.jboss.org]

Author: richard.opalka at jboss.com
Date: 2009-10-13 08:58:29 -0400 (Tue, 13 Oct 2009)
New Revision: 10901

Modified:
   stack/native/branches/ropalka/modules/core/src/main/java/org/jboss/ws/core/jaxws/spi/http/NettyRequestHandlerImpl.java
Log:
[NETTY-233][NETTY-237] fix + TODO

Modified: stack/native/branches/ropalka/modules/core/src/main/java/org/jboss/ws/core/jaxws/spi/http/NettyRequestHandlerImpl.java
===================================================================
--- stack/native/branches/ropalka/modules/core/src/main/java/org/jboss/ws/core/jaxws/spi/http/NettyRequestHandlerImpl.java	2009-10-13 11:38:20 UTC (rev 10900)
+++ stack/native/branches/ropalka/modules/core/src/main/java/org/jboss/ws/core/jaxws/spi/http/NettyRequestHandlerImpl.java	2009-10-13 12:58:29 UTC (rev 10901)
@@ -168,7 +168,7 @@
          Map<String, List<String>> responseHeaders, Channel channel) throws IOException
    {
       // Build the response object.
-      HttpResponse response = new DefaultHttpResponse(HttpVersion.HTTP_1_1, getResponseStatus(statusCode));
+      HttpResponse response = new DefaultHttpResponse(HttpVersion.HTTP_1_1, HttpResponseStatus.valueOf(statusCode));
 
       Iterator<String> iterator = responseHeaders.keySet().iterator();
       String key = null;
@@ -236,6 +236,7 @@
 
    private String removeProhibitedCharacters(String s)
    {
+      // TODO: https://jira.jboss.org/jira/browse/NETTY-237
       String retVal = s;
 
       retVal = retVal.replace('\r', ' ');
@@ -244,17 +245,4 @@
       return retVal;
    }
 
-   private HttpResponseStatus getResponseStatus(int statusCode)
-   {
-      // TODO: https://jira.jboss.org/jira/browse/NETTY-233 
-      if (statusCode == 500)
-         return HttpResponseStatus.INTERNAL_SERVER_ERROR;
-      if (statusCode == 202)
-         return HttpResponseStatus.ACCEPTED;
-      if (statusCode == 204)
-         return HttpResponseStatus.NO_CONTENT;
-
-      return HttpResponseStatus.OK;
-   }
-
 }