Thanks for the PR! Can you split it into two separate PRs? The JSONEncoder changes are logically separate from the access control changes, I think?

Codecov Report

Merging #1221 into master will decrease coverage by 0.07%.
The diff coverage is 81.81%.

@@            Coverage Diff             @@
##           master    #1221      +/-   ##
==========================================
- Coverage   86.94%   86.87%   -0.08%     
==========================================
  Files          38       38              
  Lines        2367     2369       +2     
==========================================
  Hits         2058     2058              
- Misses        309      311       +2

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4fbe57d...ebbc981. Read the comment docs.

@ianpartridge sure, I just reverted the access level changes, this one is now for the customJSONEncoder block property.

By changing a encoding strategy, we have to make sure we also change the decoding strategy. Meaning that the JSONEncoder and JSONDecoder should match.
For the test case example of a custom date strategy it had been done here
But if we think of Kitura-Kit, we have to make sure the JSONDecoder for example here matches our JSONEncoder in Kitura

@EnriqueL8 you are right, I just made a new PR for KituraKit as well.