sahvx655-wq commented on PR #411: URL: https://github.com/apache/commons-validator/pull/411#issuecomment-4806552435
Fair enough, that's a reasonable line to draw. The contract argument holds: if validate() trims and returns VALID, a caller could quite reasonably treat the string it handed in as a clean IBAN and pass it on with the whitespace still attached, which is worse than a clear rejection. Leaving normalisation to the caller keeps the status honest about exactly what was given. I'll leave this closed. Cheers for taking a look. -- 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]
