JSON requires escaped control characters

Sming/Core/Data/Format/Formatter.cpp

@@ -0,0 +1,98 @@
+/****
+ * Sming Framework Project - Open Source framework for high efficiency native ESP8266 development.
+ * Created 2015 by Skurydin Alexey
+ * http://github.com/SmingHub/Sming
+ * All files of the Sming Core are provided under the LGPL v3 license.
+ *
+ * Formatter.cpp
+ *
+ * @author mikee47 <[email protected]> Aug 2024
+ *
+ ****/
+
+#include "Formatter.h"
+
+namespace
+{
+/**
+ * @brief Get character used for standard escapes
+ * @param c Code to be escaped
+ * @retval char Corresponding character, NUL if there isn't a standard escape
+ */
+char escapeChar(char c)
+{
+	switch(c) {
+	case '\0':
+		return '0';
+	case '\'':
+		return '\'';
+	case '\"':
+		return '"';
+	case '\?':
+		return '?';
+	case '\\':
+		return '\\';
+	case '\a':
+		return 'a';
+	case '\b':
+		return 'b';
+	case '\f':
+		return 'f';
+	case '\n':
+		return 'n';
+	case '\r':
+		return 'r';
+	case '\t':
+		return 't';
+	case '\v':
+		return 'v';
+	default:
+		return '\0';
+	}
+}
+
+} // namespace
+
+namespace Format
+{
+unsigned escapeControls(String& value)
+{
+	// Count number of extra characters we'll need to insert
+	unsigned extra{0};
+	for(auto& c : value) {
+		if(escapeChar(c)) {
+			extra += 1; // "\"
+		} else if(c < 0x20) {
+			extra += 3; // "\xnn"
+		}
+	}
+	if(extra == 0) {
+		return 0;
+	}
+	auto len = value.length();
+	if(!value.setLength(len + extra)) {
+		return 0;
+	}
+	char* out = value.begin();
+	const char* in = out;
+	memmove(out + extra, in, len);
+	in += extra;
+	while(len--) {
+		auto c = *in++;
+		auto esc = escapeChar(c);
+		if(esc) {
+			*out++ = '\\';
+			*out++ = esc;
+		} else if(c < 0x20) {
+			*out++ = '\\';
+			*out++ = 'x';
+			*out++ = hexchar(uint8_t(c) >> 4);
+			*out++ = hexchar(uint8_t(c) & 0x0f);
+		} else {
+			*out++ = c;
+		}
+	}
+	return extra;
+}
+
+} // namespace Format

Sming/Core/Data/Format/Formatter.h

@@ -17,6 +17,13 @@
 
 namespace Format
 {
+/**
+ * @brief Escape standard control codes such as `\n` (below ASCII 0x20)
+ * @param value String to be modified
+ * @retval unsigned Number of control characters found and replaced
+ */
+unsigned escapeControls(String& value);
+
 /**
  * @brief Virtual class to perform format-specific String adjustments
  */

Sming/Core/Data/Format/Json.cpp

@@ -68,6 +68,7 @@ bool IsValidUtf8(const char* str, unsigned length)
  */
 void Json::escape(String& value) const
 {
+	escapeControls(value);
 	if(!IsValidUtf8(value.c_str(), value.length())) {
 		debug_w("Invalid UTF8: %s", value.c_str());
 		for(unsigned i = 0; i < value.length(); ++i) {