]> Cypherpunks repositories - gostls13.git/commit
cmd/dist: test encoding/json/... with GOEXPERIMENT=jsonv2
authorJoe Tsai <joetsai@digital-static.net>
Mon, 23 Jun 2025 23:36:25 +0000 (16:36 -0700)
committerJoseph Tsai <joetsai@digital-static.net>
Tue, 24 Jun 2025 01:21:07 +0000 (18:21 -0700)
commitf866958246556ec609b81d31376a39efe9d51a51
treed6bf62abb4cb9b8162627945ce2811baac641f13
parentf77a0aa6b6d90742932f0bf29d2f94459597331a
cmd/dist: test encoding/json/... with GOEXPERIMENT=jsonv2

This also updates wasip1_wasm to use a 8MiB stack, which is
the same stack size as what is used by go_js_wasm_exec.
The increase of stack size is necessary because the jsonv2
tests exercise that the jsonv2 and jsontext packages support
a hard limit of a maximum JSON nesting depth of 10000.
However, even with a depth limit of 10000, this still exceeds
the previously specified maximum stack size of 1 MiB.

For use of JSON with untrusted inputs in WASM,
we really need to support #56733 as there is no right answer
for the default max depth limit to use since the max wasm
stack size is determined on a per-system basis.

Updates #71845

Change-Id: I3b32c58cc9f594a5c59bb3e4b20f5e86d85d8209
Reviewed-on: https://go-review.googlesource.com/c/go/+/683575
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Johan Brandhorst-Satzkorn <johan.brandhorst@gmail.com>
Reviewed-by: Damien Neil <dneil@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
lib/wasm/go_wasip1_wasm_exec
src/cmd/dist/test.go