tree b862d1e577816261f2e5d281199b760d9b65b41a
parent 5c2b0867815f6b3540a3252d11dc01232ca0419f
author Marcel van Lohuizen <mpvl@golang.org> 1584440632 +0100
committer Marcel van Lohuizen <mpvl@golang.org> 1584519477 +0000

encoding/protobuf: define messages as closed

This solves the issue that the only way to map a
Protocol Buffer Oneof to CUE is to by using closed
structs.

This may seem counterintuitive, as the Protocol spec
defines all messages as extensible. There is nothing
to prevent the extension of a CUE defintion, however,
in that sense. Closedness exists to type check within
a single version of an API.

Mapping to closed structs is analogous to a generated
Go spec from Protobufs not allowing arbitrary fields.

encoding/openapi is adjusted accordingly.

This solves a mapping issue related from Istio.

Issue istio/istio#22100.

Change-Id: I69196de5d39e18720e151e04ef59c789d9424aa6
Reviewed-on: https://cue-review.googlesource.com/c/cue/+/5321
Reviewed-by: Jason Wang <jasonwzm@google.com>
Reviewed-by: Marcel van Lohuizen <mpvl@golang.org>