You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@fineract.apache.org by "pjfanning (via GitHub)" <gi...@apache.org> on 2026/10/14 11:54:01 UTC

[PR] FINERACT-2949: Configure polymorphic type validator on execution context mapper [fineract]

pjfanning opened a new pull request, #6649:
URL: https://github.com/apache/fineract/pull/6649

   ## Description
   
   See [FINERACT-2949](https://issues.apache.org/jira/browse/FINERACT-2949).
   
   Applies the existing execution context type allow-list as the mapper-wide `PolymorphicTypeValidator` on the Spring Batch execution context mapper, so annotation-driven type handling uses the same restrictions as default typing.
   
   ## Checklist
   
   - [x] Commit message starts with the JIRA ticket number
   - [x] Tests added to `BatchMetadataCompatibilityTest`
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
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: issues-unsubscribe@fineract.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] FINERACT-2949: Configure polymorphic type validator on execution context mapper [fineract]

Posted by "pjfanning (via GitHub)" <gi...@apache.org>.
pjfanning commented on PR #6649:
URL: https://github.com/apache/fineract/pull/6649#issuecomment-6262667705

   @galovics Good catch — agreed it should pin behaviour. Two wrinkles: `BasicPolymorphicTypeValidator` returns `INDETERMINATE` (not `DENIED`) for unmatched names and Jackson rejects after resolving the class, and a non-existent class like `com.example.Unlisted` fails with "no such class found" with or without the fix. So I've replaced the wiring test with one that deserializes a real `COBParameter` subclass outside the allow-listed packages and asserts the PTV "denied resolution" error — that one fails without the change (b845be332e).
   


-- 
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: issues-unsubscribe@fineract.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] FINERACT-2949: Configure polymorphic type validator on execution context mapper [fineract]

Posted by "adamsaghy (via GitHub)" <gi...@apache.org>.
adamsaghy merged PR #6649:
URL: https://github.com/apache/fineract/pull/6649


-- 
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: issues-unsubscribe@fineract.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org