Fix gob encoder error on unknown value

[syndicut]

When using Config() method with --module option you can get cty.unknownType inside module variables without default values and following error:

13:39:46 server.go:418: rpc: gob error encoding body: error encoding cty.Value: gob: type not registered for interface: cty.unknownType

This fix forces cty.Value{} in case of unknown value

[wata727]

Umm, I understand this bug, but this fix makes it impossible for plugins to handle variables as unknown.

Maybe gob is not a good way to transfer cty.Value. go-cty seems to support encoding/decoding with msgpack, so I think this might be a good idea, but I haven't started investigating anything yet. If you are interested, I would like to support you as much as possible.
https://pkg.go.dev/github.com/zclconf/[email protected]/cty/msgpack

[syndicut]

Umm, I understand this bug, but this fix makes it impossible for plugins to handle variables as unknown.

Maybe gob is not a good way to transfer cty.Value. go-cty seems to support encoding/decoding with msgpack, so I think this might be a good idea, but I haven't started investigating anything yet. If you are interested, I would like to support you as much as possible.
https://pkg.go.dev/github.com/zclconf/[email protected]/cty/msgpack

Ok, good point, will look at it

[syndicut]

Umm, I understand this bug, but this fix makes it impossible for plugins to handle variables as unknown.

Maybe gob is not a good way to transfer cty.Value. go-cty seems to support encoding/decoding with msgpack, so I think this might be a good idea, but I haven't started investigating anything yet. If you are interested, I would like to support you as much as possible.
https://pkg.go.dev/github.com/zclconf/[email protected]/cty/msgpack

Think those changes and terraform-linters/tflint-plugin-sdk#96 should do the trick

[wata727]

Merged terraform-linters/tflint-plugin-sdk#96. Could you update the sdk version in this PR?

[syndicut]

Merged terraform-linters/tflint-plugin-sdk#96. Could you update the sdk version in this PR?

Done!

[wata727]

👍