search

RicoSuter / NJsonSchema

JSON Schema reader, generator and validator for .NET
http://NJsonSchema.org
MIT License
1.37k stars 529 forks source link

Throw more detailed error messages when encountering Json Schema errors #1707

Open hyzx86 opened 2 months ago

hyzx86 commented 2 months ago

fixes: #1429

hyzx86 commented 2 months ago

Hi @bencyoung-Fignum I'm trying to deal with this issue, but since it's been a while I'm not sure if it still exists. Also it doesn't seem to be in the same position looking at your screenshot. Can you provide more info on this? Or maybe you pull my PR and try it.

bencyoung-Fignum commented 2 months ago

I don't think that will help me as there's no identifying names on the one that doesn't get a value. The only interesting thing about it is IsAnyType is true. However it has no properties or anything else naming it

image

RicoSuter commented 1 month ago

Is this still needed? I'm fine merging this as it's not really breaking or anything... Maybe it produces way too much text in the exception message.

hyzx86 commented 1 month ago

Maybe it produces way too much text in the exception message.

I think these texts are worthwhile. they can be somewhat easier for developers to locate the problem. After all, it has happened exceptionally 😅