Commit d511b4c
committed
build: disable the jsonv2 GOEXPERIMENT
Go 1.27 enables the jsonv2 experiment by default, reimplementing
encoding/json and making json.RawMessage an alias for jsontext.Value. The
repo inherited this from #122, which bumped the go directive from 1.26.6 to
1.27.0 -- that directive is a minimum toolchain requirement, so GOTOOLCHAIN
=auto then builds everything with go1.27 and its default experiments.
This emulator serializes customer Lambda payloads, so swapping JSON
implementation should be a deliberate, tested decision rather than a side
effect of a toolchain bump. Pin it off for now.
There is no GODEBUG opt-out -- jsonv2 is selected at compile time via build
tags -- so it must be an environment variable. Exporting it from the
Makefile covers local builds, the docker targets (which re-enter this
Makefile in the container) and CI, which invokes only make targets.
Fixes the localstack branch, red since Aug 22, without touching the
upstream test that the experiment broke.1 parent 7ec133c commit d511b4c
1 file changed
Lines changed: 3 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
| 4 | + | |
4 | 5 | | |
5 | 6 | | |
6 | 7 | | |
7 | 8 | | |
| 9 | + | |
| 10 | + | |
8 | 11 | | |
9 | 12 | | |
10 | 13 | | |
| |||
0 commit comments