]> Cypherpunks repositories - gostls13.git/commit
encoding/json: support TextUnmarshaler for map keys with string underlying types
authorCuong Manh Le <cuong.manhle.vn@gmail.com>
Thu, 10 Oct 2019 03:19:12 +0000 (10:19 +0700)
committerDaniel Martí <mvdan@mvdan.cc>
Thu, 10 Oct 2019 12:59:11 +0000 (12:59 +0000)
commitfb9af8411eaa3e38d2f72e28a305772f50042657
treef09dadcefa10828ec8c8e7364c4d1383c67748a9
parent900ebcfe4d592486dd5bc50f5e8101ba65e32948
encoding/json: support TextUnmarshaler for map keys with string underlying types

When unmarshaling to a map, the map's key type must either be a string,
an integer, or implement encoding.TextUnmarshaler. But for a user
defined type, reflect.Kind will not distinguish between the static type
and the underlying type. In:

var x MyString = "x"
t := reflect.TypeOf(x)
println(t.Kind() == reflect.String)

the Kind of x is still reflect.String, even though the static type of x
is MyString.

Moreover, checking for the map's key type is a string occurs first, so
even if the map key type MyString implements encoding.TextUnmarshaler,
it will be ignored.

To fix the bug, check for encoding.TextUnmarshaler first.

Fixes #34437

Change-Id: I780e0b084575e1dddfbb433fe03857adf71d05fb
Reviewed-on: https://go-review.googlesource.com/c/go/+/200237
Run-TryBot: Cuong Manh Le <cuong.manhle.vn@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Daniel Martí <mvdan@mvdan.cc>
src/encoding/json/decode.go
src/encoding/json/decode_test.go