Fixed snippet display as UTF-8, issue WAVE-246

8 views
Skip to first unread message

vruiz....@gmail.com

unread,
Oct 18, 2011, 6:00:40 AM10/18/11
to hear...@google.com, soren...@gmail.com, veg...@gmail.com, wave-protocol...@googlegroups.com
Reviewers: hearnden, Soren Lassen, Yuri Z.,

Please review this at http://codereview.waveprotocol.org/619001

Affected files:
M src/org/waveprotocol/box/server/rpc/SearchServlet.java


Index: src/org/waveprotocol/box/server/rpc/SearchServlet.java
===================================================================
--- a/src/org/waveprotocol/box/server/rpc/SearchServlet.java
+++ b/src/org/waveprotocol/box/server/rpc/SearchServlet.java
@@ -216,7 +216,7 @@
resp.sendError(HttpServletResponse.SC_FORBIDDEN);
} else {
resp.setStatus(HttpServletResponse.SC_OK);
- resp.setContentType("application/json");
+ resp.setContentType("application/json; charset=utf8");
// This is to make sure the fetched data is fresh - since the w3c
spec
// is rarely respected.
resp.setHeader("Cache-Control", "no-store");


veg...@gmail.com

unread,
Oct 18, 2011, 6:04:13 AM10/18/11
to vruiz....@gmail.com, hear...@google.com, soren...@gmail.com, wave-protocol...@googlegroups.com
On 2011/10/18 10:00:40, vjrj wrote:


Thanks for the patch!
Can you please submit the diff at https://reviews.apache.org/r/new/?
This way we will be able to link between the review and commit to the
new WIAB SVN repo at Apache.

http://codereview.waveprotocol.org/619001

Reply all
Reply to author
Forward
0 new messages