-
Notifications
You must be signed in to change notification settings - Fork 6
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
possible look at json edge cases #63
Comments
This looks valuable, thank you @pjfanning! For what it is worth, there have been improvements in many of these areas over past couple of years.
In addition there is some on-going active work; for example:
In general I think that lists like above could help figure out some problematic aspects, but do require sort of incremental approach: there will always some potential concerns; and conversely there is no universal one fix to plug all holes. Anyway: I think I will add above link to issue "core#637" as some more background. |
https://github.com/zio/zio-json#security has a good description of trying to fail early for malicious json inputs. I'm not sure where jackson-databind stands with problematic json as described in the link. I guess the aim would be to fail while building the JSON AST.
Apologies if jackson(-databind) already has support for these malicious formats.
The text was updated successfully, but these errors were encountered: