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

"additionalProperties: true" for struct tagged with "#[serde(deny_unknown_fields)]" #29

Closed
wants to merge 1 commit into from

Conversation

dkourilov
Copy link
Contributor

Mentioned in #27 as well as TODO in the code.

If #[serde(deny_unknown_fields)] is not specified, default behavior (as with additionalProperties: true, i.e. properties are allowed but skipped).

@dkourilov dkourilov changed the title additionalProperties: false for struct tagged with #[serde(deny_unknown_fields)] "additionalProperties: false" for struct tagged with "#[serde(deny_unknown_fields)]" May 14, 2020
@dkourilov dkourilov changed the title "additionalProperties: false" for struct tagged with "#[serde(deny_unknown_fields)]" "additionalProperties: true" for struct tagged with "#[serde(deny_unknown_fields)]" May 14, 2020
@dkourilov dkourilov closed this May 14, 2020
@dkourilov
Copy link
Contributor Author

Sorry too late, logic is messed up, I'll reopen tomorrow.

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

Successfully merging this pull request may close these issues.

1 participant