Bug 16610 - Regression in SIP2 user password handling
commit3cd0ac25fa7f3bc4aa60d2b4b24f0ddd05839606
authorKyle M Hall <kyle@bywatersolutions.com>
Fri, 27 May 2016 12:42:17 +0000 (27 12:42 +0000)
committerJulian Maurice <julian.maurice@biblibre.com>
Thu, 16 Jun 2016 07:55:13 +0000 (16 09:55 +0200)
treede6d3fef296e0ca65c9a617c80b72f5571f9539b
parent8d9a4841e4be97f53c94282f08f0e53bdaf6487f
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>
(cherry picked from commit 66c9ef12c16efeebf78e3a98be7b2ccc42d14e7a)
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
C4/SIP/ILS/Patron.pm