Brad Fitzpatrick 82cfea90ca all: fix JSON serialization under Go 1.27's finalized encoding/json/v2 ... Go 1.27 enables GOEXPERIMENT=jsonv2 by default: encoding/json is now backed by the json/v2 machinery, and github.com/go-json-experiment/json compiles as a thin alias of the standard library's encoding/json/v2. Several tag options and behaviors we relied on did not make the cut for the final Go 1.27 API, breaking tailscaled at runtime and four packages' tests. This change adapts to the final API while keeping the wire format byte-for-byte identical on all Go versions. First, the `format` tag option was demoted to experimental. Its mere presence in a struct tag now makes marshaling and unmarshaling fail at runtime. tailcfg.SSHAction.SessionDuration had `format:nano` (added in a2dc517d7 to pin the v1 representation), so on Go 1.27 any netmap containing an SSH policy failed to decode, breaking every PollNetMap. Remove the option here and in net/speedtest; time.Duration still marshals as int64 nanoseconds under encoding/json on all Go versions (Go 1.27's v1 mode sets FormatDurationAsNano by default), so old clients and servers are unaffected. Add a regression test locking in the exact wire format. Consequently, invert the cmd/vet jsontags rule: it previously required an explicit `format` tag on time.Duration fields, which is now exactly wrong. It now rejects any `format` tag option, which would have caught this bug in CI. Second, the `inline` tag option was renamed to `embed`. The standard library silently ignores `inline`, while the pinned go-json-experiment module (used on Go 1.26) only knows `inline`. Specify both options in types/prefs and logtail; each implementation ignores the option it does not know, producing identical output. Drop `inline` once we require Go 1.27. Third, encoding/json (v1) now dispatches to MarshalJSONTo and UnmarshalJSONFrom methods and its v1 options flow into nested jsonv2.MarshalEncode calls. Types whose v1 methods deliberately routed through jsonv2 for v2 semantics (types/opt.Value, the types/prefs preference types) would silently change wire format (e.g. nil slices becoming null). Pin jsonv2.DefaultOptionsV2 in their jsonv2 methods so the representation is the same regardless of the entry point. Finally, json.Marshal costs one more allocation under Go 1.27, tripping the types/logger.AsJSON alloc test. Switch its fmt.Formatter to jsonv2.MarshalWrite with explicit v1 options, which writes directly to the fmt.State: one allocation on both toolchains with unchanged output. Depaware files pick up the go-json-experiment/json/v1 options shim as a new dependency of types/logger. With this change, go test ./... passes with both Go 1.26.5 and go1.27rc2. Updates #20220 Fixes #20528 Fixes #20254 Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com> Change-Id: I694c7d57fd81e55a579c579e9be10032bca569d4		2026-07-19 09:58:50 -07:00
..
appctype	feature/conn25,types/appctype: serve active Conn25 state over localapi	2026-07-16 16:35:38 -04:00
bools	all: remove AUTHORS file and references to it	2026-01-23 15:49:45 -08:00
dnstype	cmd/vet: add subtestnames analyzer; fix all existing violations	2026-04-05 15:52:51 -07:00
empty	all: remove AUTHORS file and references to it	2026-01-23 15:49:45 -08:00
events	wgengine/magicsock,control/controlclient: do not overwrite discokey with old key (#18606)	2026-03-20 08:56:27 -04:00
flagtype	all: remove AUTHORS file and references to it	2026-01-23 15:49:45 -08:00
geo	types/geo: add support for ScalarMarshaler and ScalarUnmarshaler (#20158)	2026-06-16 16:36:43 -07:00
iox	all: remove AUTHORS file and references to it	2026-01-23 15:49:45 -08:00
ipproto	all: use Go 1.26 things, run most gofix modernizers	2026-03-06 13:32:03 -08:00
jsonx	types/ptr: deprecate ptr.To, use Go 1.26 new	2026-03-05 20:13:18 -08:00
key	all: fix more typos caused by unnecessary repetition	2026-07-06 10:36:17 +01:00
lazy	all: use Go 1.26 things, run most gofix modernizers	2026-03-06 13:32:03 -08:00
logger	all: fix JSON serialization under Go 1.27's finalized encoding/json/v2	2026-07-19 09:58:50 -07:00
logid	all: remove AUTHORS file and references to it	2026-01-23 15:49:45 -08:00
mapx	net/tsdial, ipn/ipnlocal: stop using netmap.NetworkMap in Dialer	2026-06-24 13:14:45 -07:00
netlogfunc	all: remove AUTHORS file and references to it	2026-01-23 15:49:45 -08:00
netlogtype	all: remove AUTHORS file and references to it	2026-01-23 15:49:45 -08:00
netmap	ipn/ipnlocal, types/netmap, tsnet: filter unsigned peers on delta path	2026-06-23 08:12:36 -07:00
nettype	all: remove AUTHORS file and references to it	2026-01-23 15:49:45 -08:00
opt	all: fix JSON serialization under Go 1.27's finalized encoding/json/v2	2026-07-19 09:58:50 -07:00
persist	all: update a few more references to network/tailnet lock	2026-05-28 16:44:16 +01:00
prefs	all: fix JSON serialization under Go 1.27's finalized encoding/json/v2	2026-07-19 09:58:50 -07:00
preftype	all: remove AUTHORS file and references to it	2026-01-23 15:49:45 -08:00
ptr	types/ptr: deprecate ptr.To, use Go 1.26 new	2026-03-05 20:13:18 -08:00
result	all: remove AUTHORS file and references to it	2026-01-23 15:49:45 -08:00
structs	all: remove AUTHORS file and references to it	2026-01-23 15:49:45 -08:00
tkatype	all: remove AUTHORS file and references to it	2026-01-23 15:49:45 -08:00
views	all: use Go 1.26 things, run most gofix modernizers	2026-03-06 13:32:03 -08:00