Author: lindner
Date: Thu Oct 15 07:30:25 2009
New Revision: 825418

URL: http://svn.apache.org/viewvc?rev=825418&view=rev
Log:
SHINDIG-1186 | Patch from Mark Weitzel | /gadgets/metadata does not handle 
Transfer-Encoding according to the HTTP/1.1 spec

Modified:
    
incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/RpcServlet.java
    
incubator/shindig/trunk/java/server/src/test/java/org/apache/shindig/server/endtoend/EndToEndTest.java

Modified: 
incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/RpcServlet.java
URL: 
http://svn.apache.org/viewvc/incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/RpcServlet.java?rev=825418&r1=825417&r2=825418&view=diff
==============================================================================
--- 
incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/RpcServlet.java
 (original)
+++ 
incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/RpcServlet.java
 Thu Oct 15 07:30:25 2009
@@ -80,31 +80,12 @@
         ? callbackValue + '(' + result.getOutput() + ')'
         : result.getOutput());
   }
-
+  
   @Override
   protected void doPost(HttpServletRequest request, HttpServletResponse 
response)
       throws IOException {
-
-    int length = request.getContentLength();
-    if (length <= 0) {
-      logger.info("No Content-Length specified.");
-      response.setStatus(HttpServletResponse.SC_LENGTH_REQUIRED);
-      return;
-    }
-    if (length > POST_REQUEST_MAX_SIZE) {
-      logger.info("Request size too large: " + length);
-      response.setStatus(HttpServletResponse.SC_REQUEST_ENTITY_TOO_LARGE);
-      return;
-    }
-
     ServletInputStream is = request.getInputStream();
     byte[] body = IOUtils.toByteArray(is);
-    if (body.length != length) {
-      logger.info("Wrong size. Length: " + length + " real: " + body.length);
-      response.setStatus(HttpServletResponse.SC_BAD_REQUEST);
-      return;
-    }
-
     Result result = process(request, response, body);
     response.getWriter().write(result.getOutput());
   }

Modified: 
incubator/shindig/trunk/java/server/src/test/java/org/apache/shindig/server/endtoend/EndToEndTest.java
URL: 
http://svn.apache.org/viewvc/incubator/shindig/trunk/java/server/src/test/java/org/apache/shindig/server/endtoend/EndToEndTest.java?rev=825418&r1=825417&r2=825418&view=diff
==============================================================================
--- 
incubator/shindig/trunk/java/server/src/test/java/org/apache/shindig/server/endtoend/EndToEndTest.java
 (original)
+++ 
incubator/shindig/trunk/java/server/src/test/java/org/apache/shindig/server/endtoend/EndToEndTest.java
 Thu Oct 15 07:30:25 2009
@@ -254,7 +254,7 @@
     
     assertEquals("45", page.getElementById("sum").getTextContent().trim());
     assertEquals("25", page.getElementById("max").getTextContent().trim());
-}
+  }
 
   @Test
   public void testTemplateLibrary() throws Exception {


Reply via email to