Marshal() generate invalid JSON about go-cty HOT 2 CLOSED

Thanks for reporting this, @denis256!

With the code you showed I would've expected the call to ctyjson.Marshal to fail, because there's no way to serialize an unknown value in JSON. Instead, it seems to have succeeded but just skipped serializing the invalid value.

I can reproduce this behavior in the Go playground: https://go.dev/play/p/Z_ztLVhfE4Q

The correct behavior here would be for ctyjson.Marshal to return an error, and therefore your caller would fail with log.Fatal.

I will try to debug and fix this at some point, but note that it's typically incorrect to pass unknown values into the JSON encoder and so you could potentially avoid this problem today by screening for unknown values first. For example, you could call valMapAsCty.IsWhollyKnown() first and return an error early if it returns false, since once corrected that call would return an error in that situation anyway.

from go-cty.

Thanks again for reporting this!

I've just fixed this in 4a34c33 and intend to make a new patch release containing that fix soon.

from go-cty.