fix(isIBAN): adjusting Ireland and Palestine IBAN regex by Tarasz57 · Pull Request #2518 · validatorjs/validator.js
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kindly correct me, but If I understand this correctly, the Palestine RegExp would be wrong here (the last part):
On page 6 of the supplied PDF we can see the convention:

The supplied PDF says the IBAN structure is PS2!n4!a21!c
The regex here shows /^(PS[0-9]{2})[A-Z]{4}\d{21}$/
So I would argue, that the \d{21} part should actually be [a-zA-Z0-9] or [A-Z0-9] (since we don't care about case, I think).
what do you think?