Bug 16610 - Regression in SIP2 user password handling
authorKyle M Hall <kyle@bywatersolutions.com>
Fri, 27 May 2016 12:42:17 +0000 (12:42 +0000)
committerFrédéric Demians <f.demians@tamil.fr>
Wed, 15 Jun 2016 05:35:58 +0000 (07:35 +0200)
commit66c9ef12c16efeebf78e3a98be7b2ccc42d14e7a
tree2f3f1ddbe51d844c48b2a6c3155e2b420b91fc70
parent9f935b384fd01c863af1de195f4fe256450f4a46
Bug 16610 - Regression in SIP2 user password handling

Previous to bug 14507, SIP2 only did internal authentication. A change
to the way we check empty passwords has caused any empty password to
send back a CQ of Y. Previous to that patch set, a CQ of Y would only be
sent back of the patron password column was NULL. Now, an empty AD field
*always* returns a CQ of Y.

Test Plan:
1) Send a patron information request with an empty AD field
   Note: You must send the AD field or you won't get back a CQ field
2) Note you get back a CQ of Y
3) Apply this patch
4) Repeat step 1
5) Note you now get back a CQ of N

Signed-off-by: Trent Roby <troby@bclib.info>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Brendan Gallagher <brendan@bywatersolutions.com>
(cherry picked from commit 71f0370ddde0d8b8e2a666a091081186fb6531e8)
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
C4/SIP/ILS/Patron.pm