-
Notifications
You must be signed in to change notification settings - Fork 14
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
Improved validation error handling #1
Comments
This has mostly been addressed. Further refinements will be made to the validation error reporting. |
Re-opening as the library could use a re-write of the error handling mechanisms for both parsing and validation. Will likely leverage thiserror. |
Decided on https://github.com/yaahc/displaydoc due to its support for |
@tomachristian would love your input on how I can improve the error handling mechanisms in the validator. Some of this is still dependent on the ratification of JSONPath as a formal standard. It would also be good to get some of the community's input on how CDDL should be traversed for solid error reporting. CC @cabo for input too |
Which kinds of errors are you looking at?
Right now, I'm very interested in the latter. |
Thanks @cabo. Interested in the validation failures specifically. And yea, at the moment, this crate is just presenting all of the failures in the case of type choices where none of the options are valid. It would be good to have some sort of JSONPath-like mechanism for pin-pointing where in the CDDL the validation failure occurred. |
All errors are currently
Box
'ed with little actionable information. The library should provide for better error handling mechanisms for lexing, parsing and validation.The text was updated successfully, but these errors were encountered: