-
Notifications
You must be signed in to change notification settings - Fork 209
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
Need to handle additionalProperties in JSON serialization #1462
Comments
Where does this come up? |
Mostly services that have a “map but with some well-known entries” pattern. |
We're still interested in doing this. |
Have we actually already done this? We should double-check |
We haven't done this, but would be good to do. |
No change from above |
Wondering if there's a way to know which services these are? Maybe we should take a look and get an understanding of how large the problem is to understand how important this is to fix? |
Haven't had time to do the above checking yet, but it also doesn't seem to be a major problem for users currently. |
When serializing an
ObjectType
that has anadditionalProperties
field to JSON, we need to flatten the properties inside it into the “parent” type. I’m not sure this is even possible with the built-in JSON Go serialization library, so we might need to look elsewhere.The text was updated successfully, but these errors were encountered: