chore: make fields accessible to enforce trustable lookup

jdereg / json-io

Convert Java to JSON. Convert JSON to Java. Pretty print JSON. Java JSON serializer. Deep copy Java object graphs.

Apache License 2.0

331 stars 116 forks source link

chore: make fields accessible to enforce trustable lookup #292

Closed ozhelezniak-talend closed 2 months ago

ozhelezniak-talend commented 2 months ago

@jdereg I had in mind smth like that

jdereg commented 2 months ago

I merged your latest PR. Would you mind updating user-guide-readOptions.md to document addInjectorFactory()?

Also, so it doesn’t have to be specified each time in the ReadOptionsBuilder, I would think we would want an “addPermanentInjectorFactory()” method too (and document for that as well). The addPermanent* APIs are nice so that you do not have to modified the ReadOptionsBuilder each time, if you want the same options always (at least within in a given service).