From 1e48ca7020adf13e0fa4a6c2053f210b869bfa24 Mon Sep 17 00:00:00 2001 From: Joe Tsai Date: Wed, 9 Jul 2025 17:08:49 -0700 Subject: encoding/json: remove legacy option to EscapeInvalidUTF8 In the presence of invalid UTF-8, the AllowInvalidUTF8 option allows such bytes to be present, but silently mangles them using the Unicode replacement character. The v2 default is to emit the replacement character verbatim (which is valid UTF-8 and exactly what it is for). However, the v1 behavior has historically been to emit the escaped form of the replacement character. This behavior was introduced in https://go.dev/cl/11211045 where the documentation says that it is: replacing invalid bytes with the Unicode replacement rune U+FFFD but the implementation actually replaces it with the escaped form of the Unicode replacement rune. Given that the documentation differs from the implementation, the actual behavior is likely an oversight. Given how esoteric of behavior this is, we change the v1in2 behavior to avoid the unnecesary escaping and drop support for EscapeInvalidUTF8. This does not violate the Go compatibility agreement since we do not document what the exact syntactic output is. Also, there has already been prior precedence for changing the output: * [encoding/json: encode \b and \f as '\b' and '\f' in JSON strings](https://go.dev/cl/521675) * [encoding/json: encode \n in strings as "\n", not "\u000A"](https://go.dev/cl/4678046) * [encoding/json: encode \t as \t instead of \u0009](https://go.dev/cl/162340043) * [encoding/json: use standard ES6 formatting for numbers during marshal](https://go.dev/cl/30371) Fixes #74551 Change-Id: Ib59a873c44713d302f1f6ab103ffba2520d63276 Reviewed-on: https://go-review.googlesource.com/c/go/+/687116 Auto-Submit: Joseph Tsai LUCI-TryBot-Result: Go LUCI Reviewed-by: Damien Neil Reviewed-by: Johan Brandhorst-Satzkorn Reviewed-by: Carlos Amedee --- src/encoding/json/v2_diff_test.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/encoding/json/v2_diff_test.go') diff --git a/src/encoding/json/v2_diff_test.go b/src/encoding/json/v2_diff_test.go index 7a561732f4..9d0798ed1d 100644 --- a/src/encoding/json/v2_diff_test.go +++ b/src/encoding/json/v2_diff_test.go @@ -786,8 +786,8 @@ func TestInvalidUTF8(t *testing.T) { switch { case json.Version == "v1" && err != nil: t.Fatalf("json.Marshal error: %v", err) - case json.Version == "v1" && string(got) != `"\ufffd"`: - t.Fatalf(`json.Marshal = %s, want "\ufffd"`, got) + case json.Version == "v1" && string(got) != "\"\ufffd\"": + t.Fatalf(`json.Marshal = %s, want %q`, got, "\ufffd") case json.Version == "v2" && err == nil: t.Fatal("json.Marshal error is nil, want non-nil") } -- cgit v1.3-5-g9baa