-
Notifications
You must be signed in to change notification settings - Fork 136
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
dependencies
property should not be an Array but Object
#119
Comments
@chriskessel Unfortunately I don't know enough about JSON Schema to really answer. This would be good to (try to) discuss on FWIW, this module works on v3 of JSON Schema spec, and it does not seem possible to create something that works on both. There exists a v4 module too (I don't have link handy but should be googlable). |
FWIW, v3 and v4 have the same wording about the dependencies field being an object. |
I don't doubt your statement, just pointing out that for any follow-up work, filing the issue itself probably won't lead to much changes. |
Why? Is jackson jsonSchema a dead project? |
@chriskessel I haven't seen anyone besides me reading issues here; and I did not write this module. I have tried to make occasional changes, but do not have use for JSON Schema myself (nor deep knowledge). There may be others interested in this module, but if so they are best reached via jackson mailing lists. So this is not an active project by any means. It seems to be used quite widely, however. |
dependencies
property should not be an Array but Object
Fixed as per merged #120; will be in 2.9.0. |
The generated schema for my object creates this:
If I run that through the schema validator, it complains that dependencies should be an object, not an array. Looking at the v3 and v4 specs, they do call out this field as an object.
This is a bug in the schema generator? Or is there a spec addendum that I missed?
(A customer pointed out that our schema had this error and we use Jackson JsonSchema 2.7.0 to generate the schema).
The text was updated successfully, but these errors were encountered: