You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@shindig.apache.org by et...@apache.org on 2008/05/21 20:55:52 UTC
svn commit: r658810 -
/incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/JsServlet.java
Author: etnu
Date: Wed May 21 11:55:51 2008
New Revision: 658810
URL: http://svn.apache.org/viewvc?rev=658810&view=rev
Log:
Applied SHINDIG-292 to fix potential content-length bug in JsServlet when the default encoding is not single byte compatible.
Modified:
incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/JsServlet.java
Modified: incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/JsServlet.java
URL: http://svn.apache.org/viewvc/incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/JsServlet.java?rev=658810&r1=658809&r2=658810&view=diff
==============================================================================
--- incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/JsServlet.java (original)
+++ incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/JsServlet.java Wed May 21 11:55:51 2008
@@ -127,7 +127,8 @@
HttpUtil.setCachingHeaders(resp, 60 * 60);
}
resp.setContentType("text/javascript; charset=utf-8");
- resp.setContentLength(jsData.length());
- resp.getOutputStream().write(jsData.toString().getBytes());
+ byte[] response = jsData.toString().getBytes("UTF-8");
+ resp.setContentLength(response.length);
+ resp.getOutputStream().write(response);
}
}