Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Broken serialization of escaped quotes #191

Closed
mfojtik opened this issue Oct 31, 2017 · 0 comments
Closed

Broken serialization of escaped quotes #191

mfojtik opened this issue Oct 31, 2017 · 0 comments

Comments

@mfojtik
Copy link

mfojtik commented Oct 31, 2017

In OpenShift, when I run the serialization test on the Kubernetes 1.8.x branch, we hit this error:

roundtrip.go:325: Image: v1.Image: DockerImageMetadata: readObjectFieldAsBytes: expect : after object field, parsing 1299 ...s":{" k7\"... at {"kind":"Image","apiVersion":"v1","metadata":{"name":"V鯙偎間\"c谴ʇ抜mn","generateName":"r","selfLink":"漋y翑c殀","uid":"4¯ȭɀ头蓿ɲė","resourceVersion":"15870568406115791856","creationTimestamp":null},"dockerImageReference":"* 鸅@gTçđ苕侒ô","dockerImageMetadata":{"kind":"DockerImage","apiVersion":"pre012","id":"P","parent":"\\.ë\u003eɬK衣櫕柃%箉扥ļ'ȃ#","comment":"=Ĭ具Rǎʂȣ蕅娪9`宓À)懘抏Ȣ","created":"1982-10-03T23:23:30Z","container":"丂ƫ´ʌɔȾK1^秗r戧șDNl,","container_config":{"Hostname":"癡鑡ĕʜǦ\u0026*s鴏鵆閶ȑ鈖H鮬H劔","Domainname":"$N(ȅ擷僂×暪","User":"鮪ƻĽƦ霳FLÁ鱝LtF恽b\u003c{鉥h","Memory":-5231482430376515094,"MemorySwap":5882410654260929314,"CpuShares":1736859303208040165,"Cpuset":"Ơ«ôŘ僰ƶ","AttachStdin":true,"AttachStderr":true,"Tty":true,"StdinOnce":true,"Cmd":["d0"],"Dns":["Ø钂Ŝ牙菫ʩŭ殻l"],"Image":"錭4-0ÂQ侬娵ŵ","VolumesFrom":"跬ɽ!HʡǗ醋ȥ幊[騕瞂PƟŪ鍉瓹泞ɞ","WorkingDir":"!)5r"},"docker_version":"aHč2ķ錭","author":"喚ʤ疞ĭ-瓞闤ɦƐPŕ碶{|鴆","config":{"Hostname":"W/t鶒`","Domainname":"拔瞇骦貌Ɲ","User":"-2*%Ģ俠","Memory":-6875146354939209016,"MemorySwap":-3830464167971486329,"CpuShares":-3295990050702446453,"Cpuset":"¹祅噁u","ExposedPorts":{" k7\"g桛/WqȲ庖":

It suggests that handling of \" was not done properly. I tried to use the jsoniter.ConfigCompatibleWithStandardLibrary but got same result.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant