]> git.koha-community.org Git - koha.git/commit
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)
committerBrendan Gallagher <brendan@bywatersolutions.com>
Fri, 3 Jun 2016 06:38:30 +0000 (06:38 +0000)
commit71f0370ddde0d8b8e2a666a091081186fb6531e8
tree2f3f1ddbe51d844c48b2a6c3155e2b420b91fc70
parentcbdd9194eb3359d517fbda9a50a02fbc375bf5dd
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>
C4/SIP/ILS/Patron.pm