g0w6y commented on PR #1774: URL: https://github.com/apache/struts/pull/1774#issuecomment-4969396460
Addressed all three notes in 3b184bf. Added the array creator param test (testArrayOfRecordsAsCreatorParam) with a WithArray fixture, so the collection matrix now covers List, Map and array. Also added the two clarifying comments you suggested: one in AuthorizingValueDeserializer noting that a redacted primitive component falls back to the type default when FAIL_ON_NULL_FOR_PRIMITIVES is off, and one in RedactionAwareDeserializer noting that a redaction co located with an unrelated mapping error is folded into object dropped as a deliberate fail closed choice. Full ParameterAuthorizingModuleTest suite passes locally (24 tests). This change lives in the REST Jackson plugin and is independent of the JSON plugin per request refactor in #1782, so it can land on its own. Ready whenever you are. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
