cmandib commented on PR #6670:
URL: https://github.com/apache/fineract/pull/6670#issuecomment-6398872630
While working on FINERACT-2961 I found a regression in this PR. Password
history rows saved before the 2019 switch to prefixed hashes (V353,
FINERACT-724) have no `{id}` prefix, so `matches()` throws
`IllegalArgumentException` on them and the password change fails when the reuse
check is on. The old `equals()` never matched those rows, so the new commit
skips them, which keeps that behaviour. There's a unit test for it.
--
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]
