Fix SystemTextJson read only properties handling

RicoSuter / NJsonSchema

JSON Schema reader, generator and validator for .NET

http://NJsonSchema.org

MIT License

1.37k stars 529 forks source link

Fix SystemTextJson read only properties handling #1697

Closed mereth closed 2 months ago

mereth commented 4 months ago

The SystemTextJsonReflectionService didn't properly handle when a setter (or getter) was missing. As a result read-only private properties and read-only static properties were not ignored when building type schema via reflection.

Fixes #1694

Erwinvandervalk commented 2 months ago

I just ran into the same problem and wanted to raise the same PR as a fix :) Nice job! Hope it get's merged soon!

RicoSuter commented 2 months ago

thanks for the PR.