mcc0nnell opened a new pull request, #133: URL: https://github.com/apache/fineract-consumer-facing/pull/133
Description StepUpTokenService currently exposes multiple actionFingerprint overloads that serialize action parameters through separate code paths. The current transfer initiate/confirm call sites are aligned, but keeping several overloads makes it possible for future callers to represent the same logical step-up action differently and therefore produce different fingerprints. This change: * replaces the duplicate overloads with a single canonical fingerprint path * keeps BigDecimal normalization centralized using stripTrailingZeros().toPlainString() * preserves the fingerprint values produced by the existing call shapes * adds regression coverage for account IDs, account type, amounts, and string-only action parameters This is intentionally behavior-preserving for current callers. The goal is to remove an API footgun and make step-up fingerprint generation deterministic through one implementation path. Addresses #113. -- 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]
