Builder evergreen-master-ubuntu-12.04-x86 Build #1208

Results:

Build successful

SourceStamp:

BuildSlave:

Reason:

scheduler

Steps and Logfiles:

1. git update ( 0 secs )
   1. stdio
2. Bootstrap via autogen './autogen.sh' skipped ( 0 secs )
   1. - no logs -
3. Bootstrap via autoreconf 'autoreconf -f ...' ( 9 secs )
   1. stdio
4. configure configure ( 7 secs )
   1. stdio
5. compile compile ( 1 secs )
   1. stdio
6. test test 848 tests 848 passed ( 13 secs )
   1. stdio
7. pylint pylint convention=203 error=1 fatal=14 refactor=6 warning=21 failed ( 1 secs )
   1. stdio
   2. convention
   3. error
   4. fatal
   5. refactor
   6. warning

Build Properties:

Forced Build Properties:

Responsible Users:

1. Galen Charlton

   gmcohnoyoudont@esilibrary.com

Timing:

All Changes:

:

1. Change #3332

   Category	None
   Changed by	Galen Charlton <gmcohnoyoudont@esilibrary.com>
   Changed at	Mon 09 May 2016 16:40:52
   Repository	git://git.evergreen-ils.org/Evergreen.git
   Project	Evergreen
   Branch	master
   Revision	82c0202c1e3ba3066fd831fef6b447de11c5c15d

   Comments

   LP#1579225: add live_t regression test
   Signed-off-by: Galen Charlton <gmc@esilibrary.com>

   Changed files

   • Open-ILS/src/perlmods/live_t/17-lp1579225_new_patron_passwords.t

2. Change #3333

   Category	None
   Changed by	Galen Charlton <gmcohnoyoudont@esilibrary.com>
   Changed at	Mon 09 May 2016 16:41:22
   Repository	git://git.evergreen-ils.org/Evergreen.git
   Project	Evergreen
   Branch	master
   Revision	9289743a74491deb7f4b960340eda2ff14ede493

   Comments

   LP#1579225: fix handling of passwords in patron registration
   This patch improves how the new password hashing is invoked
   by open-ils.actor.patron.update; in particular, it fixes
   a problem whereby newly registered patrons could not
   log in.  It also fixes other issues:
   
   - actor.usr.passwd would be set to an MD5 of the password
     for new users, vitiating the strong hashes in actor.passwd
   - certain types of updates via patron registration, such as
     adding or deleting addresses, could result in the patron's
     password getting doubly-hashed, thereby locking them out
     of their account.
   
   To test
   -------
   [1] Register a new patron; verify that they can log in.
   [2] Edit an existing patron and change their password; verify
       that they can log in.
   [3] Edit an existing patron but do NOT change their password;
       verify that they can still log in.
   [4] Inspect the actor.usr rows for these patrons and verify
       that actor.usr.passwd is set to the value MD5(''), not
       the MD5 of their password.
   
   Signed-off-by: Galen Charlton <gmc@esilibrary.com>
   Signed-off-by: Dan Wells <dbw2@calvin.edu>
   Signed-off-by: Mike Rylander <mrylander@gmail.com>
   Signed-off-by: Kathy Lussier <klussier@masslnc.org>
   Signed-off-by: Galen Charlton <gmc@esilibrary.com>

   Changed files

   • Open-ILS/src/perlmods/lib/OpenILS/Application/Actor.pm