From 5866e64ebcc4d6a80fb3403b6a38317fe5a52e74 Mon Sep 17 00:00:00 2001 From: hlaaf Date: Tue, 29 May 2018 02:15:37 +0300 Subject: fix #7881, control characters in json output (#7887) * fix #7881, control characters in json output * Add test for control characters in json --- lib/pure/json.nim | 3 +++ 1 file changed, 3 insertions(+) diff --git a/lib/pure/json.nim b/lib/pure/json.nim index 2bb830bcb..e7ad5bd5a 100644 --- a/lib/pure/json.nim +++ b/lib/pure/json.nim @@ -558,6 +558,8 @@ proc escapeJson*(s: string; result: var string) = of '\t': result.add("\\t") of '\r': result.add("\\r") of '"': result.add("\\\"") + of '\0'..'\7': result.add("\\u000" & $ord(c)) + of '\14'..'\31': result.add("\\u00" & $ord(c)) of '\\': result.add("\\\\") else: result.add(c) result.add("\"") @@ -1581,6 +1583,7 @@ when isMainModule: doAssert(parsed2{"repository", "description"}.str=="IRC Library for Haskell", "Couldn't fetch via multiply nested key using {}") doAssert escapeJson("\10Foo🎃barÄ") == "\"\\nFoo🎃barÄ\"" + doAssert escapeJson("\0\7\20") == "\"\\u0000\\u0007\\u0020\"" # for #7887 # Test with extra data when not defined(js): -- cgit 1.4.1-2-gfad0