Bug 21997: SIP patron information requests can lock patron out of account
authorKyle M Hall <kyle@bywatersolutions.com>
Thu, 13 Dec 2018 19:56:22 +0000 (14:56 -0500)
committerroot <root@f1ebe1bec408>
Fri, 22 Feb 2019 13:09:07 +0000 (13:09 +0000)
commitfac2c172143b31255767684e4b22c0ba1ae0aaaf
tree65c2738b32fb5bc1d3cdefffacb6e5a53b74bdf7
parentd0a6469ad58ccd6067bdfbb6cfb0e28dabc5862a
Bug 21997: SIP patron information requests can lock patron out of account

Many SIP services send an empty password field (AD). Even if allow_empty_passwords is enabled for the given SIP account, this empty password is run though Koha's password checker which increments the number of login attempts for a patron. Thus repeated patron information requests can lock a patron out! Empty password fields in SIP should not call for a password check if allow_empty_passwords is enabled.

Test Plan:
1) Enable a patron password attempt with a limit of 3
2) Send 4 patron information requests with an empty AD field
3) Note the patron's account is now locked
4) Apply this patch
5) Repeat step 2 with a different patron
6) Note the patron's account does not get locked!

Signed-off-by: Charles Farmer <charles.farmer@inLibro.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
C4/SIP/Sip/MsgType.pm