You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@shindig.apache.org by ch...@apache.org on 2010/04/07 01:29:15 UTC

svn commit: r931371 - /shindig/trunk/java/common/src/main/java/org/apache/shindig/protocol/JsonRpcServlet.java

Author: chirag
Date: Tue Apr  6 23:29:14 2010
New Revision: 931371

URL: http://svn.apache.org/viewvc?rev=931371&view=rev
Log:
Set correct http status codes in JsonRpcServlet
Code review: http://codereview.appspot.com/638046/show


Modified:
    shindig/trunk/java/common/src/main/java/org/apache/shindig/protocol/JsonRpcServlet.java

Modified: shindig/trunk/java/common/src/main/java/org/apache/shindig/protocol/JsonRpcServlet.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/common/src/main/java/org/apache/shindig/protocol/JsonRpcServlet.java?rev=931371&r1=931370&r2=931371&view=diff
==============================================================================
--- shindig/trunk/java/common/src/main/java/org/apache/shindig/protocol/JsonRpcServlet.java (original)
+++ shindig/trunk/java/common/src/main/java/org/apache/shindig/protocol/JsonRpcServlet.java Tue Apr  6 23:29:14 2010
@@ -289,6 +289,8 @@ public class JsonRpcServlet extends ApiS
   protected void sendError(HttpServletResponse servletResponse, ResponseItem responseItem)
       throws IOException {
     jsonConverter.append(servletResponse.getWriter(), getErrorJson(responseItem));
+
+    servletResponse.setStatus(responseItem.getErrorCode());
   }
 
   private void sendBadRequest(Throwable t, HttpServletResponse response) throws IOException {