From: Ben Darnell <ben@bendarnell.com>
Date: Sun, 26 Jun 2011 02:02:55 +0000 (-0700)
Subject: Accept utf8-encoded byte strings in json_encode
X-Git-Tag: v2.1.0~153
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=81dea5fb759fb15b41afcb60ea0566759716187b;p=thirdparty%2Ftornado.git

Accept utf8-encoded byte strings in json_encode
---

diff --git a/tornado/escape.py b/tornado/escape.py
index fd686bb4d..44fe9368d 100644
--- a/tornado/escape.py
+++ b/tornado/escape.py
@@ -79,7 +79,7 @@ def json_encode(value):
     # the javscript.  Some json libraries do this escaping by default,
     # although python's standard library does not, so we do it here.
     # http://stackoverflow.com/questions/1580647/json-why-are-forward-slashes-escaped
-    return _json_encode(value).replace("</", "<\\/")
+    return _json_encode(recursive_unicode(value)).replace("</", "<\\/")
 
 
 def json_decode(value):
diff --git a/tornado/test/escape_test.py b/tornado/test/escape_test.py
index 5904a540c..42ba50bbf 100644
--- a/tornado/test/escape_test.py
+++ b/tornado/test/escape_test.py
@@ -3,7 +3,7 @@
 import tornado.escape
 import unittest
 
-from tornado.escape import utf8, xhtml_escape, xhtml_unescape, url_escape, url_unescape, to_unicode, json_decode
+from tornado.escape import utf8, xhtml_escape, xhtml_unescape, url_escape, url_unescape, to_unicode, json_decode, json_encode
 from tornado.util import b
 
 linkify_tests = [
@@ -180,3 +180,10 @@ class EscapeTestCase(unittest.TestCase):
 
         # Non-ascii bytes are interpreted as utf8
         self.assertEqual(json_decode(utf8(u'"\u00e9"')), u"\u00e9")
+
+    def test_json_encode(self):
+        # json deals with strings, not bytes, but our encoding function should
+        # accept bytes as well as long as they are utf8.
+        self.assertEqual(json_decode(json_encode(u"\u00e9")), u"\u00e9")
+        self.assertEqual(json_decode(json_encode(utf8(u"\u00e9"))), u"\u00e9")
+        self.assertRaises(UnicodeDecodeError, json_encode, b("\xe9"))