Bug 12367

Summary: Import patrons still says "MD5 hash"
Product: Koha Reporter: Magnus Enger <magnus>
Component: ToolsAssignee: Magnus Enger <magnus>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: trivial    
Priority: P5 - low CC: jonathan.druart, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: String patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 9611    
Bug Blocks:    
Attachments: Bug 12367 - Import patrons still says "MD5 hash"
Bug 12367 - Import patrons still says "MD5 hash"
Bug 12367 - Import patrons still says "MD5 hash"

Description Magnus Enger 2014-06-05 07:47:33 UTC
The password hashing algorithm was changed in Bug 9611, but on Tools > Import patrons, in the text on the right hand side, it still says:

"'password' should be stored in plaintext, and will be converted to a MD5 hash (if your passwords are already encrypted, talk to your system administrator about options)."

This has no practical effect, of course, but to someone evaluating Koha it might give the false impression that password security is lower than it really is.
Comment 1 Magnus Enger 2014-06-05 08:02:30 UTC Comment hidden (obsolete)
Comment 2 David Cook 2014-06-06 05:02:57 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2014-06-06 13:45:26 UTC
Created attachment 28700 [details] [review]
Bug 12367 - Import patrons still says "MD5 hash"

The password hashing algorithm was changed in Bug 9611,
but on Tools > Import patrons, in the text on the right
hand side, it still says:

"'password' should be stored in plaintext, and will be
converted to a MD5 hash"

This has no practical effect, of course, but to someone
evaluating Koha it might give the false impression that
password security is lower than it really is.

To test:
- Look at Tools > Import patrons and verify that it says
  "a MD5 hash"
- Also look at the help page and see the same text
- Apply the patch
- Check that both the tool and the help now says
  "a Bcrypt hash"

I'll do a patch for the docs too.

Signed-off-by: David Cook <dcook@prosentient.com.au>

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 4 Tomás Cohen Arazi 2014-06-21 22:18:36 UTC
Patch pushed to master.

Thanks Magnus!