Opened 7 years ago

Closed 3 years ago

## #1283 closed enhancement (wontfix)

Reported by: Owned by: sto brad normal DbAuthPlugin normal 0.10

### Description

It would be nice to support more than one password_hash on this plugin,
as the default one used on the 0.10 version of
the plugin can't be used to validate users with apache 2.2 and
mod_authn_dbd.

As I needed the functionality I've added support for crypt passwords; I
know that crypt is not the best option, but at least I can move
passwords between different authentication systems without users noticing
it, that is, I can use them on the htpasswd file, on an LDAP server
or on the /etc/shadow files of a UNIX machine.

checks if the value is crypt when generating new passwords (if it is
not, it uses the previous SHA-1 hash). When checking passwords the patch
tries the three authentication methods (cleartext, SHA-1 and
crypt), basically to keep the system backwards compatible.

I've looked at the simple branch, but as I'm
using the 0.10 version and have not tested the simple version I have not
patched that code, but it should be easy to do.

On a quick review I've seen that on the new version only one hash method
is used when verifying passwords and that the keyword used to choose the
hash method is algorithm, but the code to support crypt is easy to
add, if you are interested I can send you a patch for the simple version
also.

### Changed 7 years ago by sto

Reviewed crypt support patch

### comment:1 Changed 7 years ago by anonymous

Yesterday I tried to add a note about the updated patch, but the
disks were full and I could not do it, so I do it today.

I've uploaded an updated version of the patch, now the CRYPT password is
stored on the Database using the prefix CRYPT:, that way the user can't
use the hashed password as a cleartext one.

mod_authn_dbd
the AuthDBDUserPWQuery has to remove the prefix, but that is easy to do
in SQL.

For PostgreSQL the query can be something like:

AuthDBDUserPWQuery "SELECT substr(password,7) FROM trac_users WHERE username = %s"


For SQLite I believe that the substring length is
also needed:

AuthDBDUserPWQuery "SELECT substr(password,7,length(password)) FROM trac_users WHERE username = %s"


### comment:2 Changed 3 years ago by rjollos

• Resolution set to wontfix
• Status changed from new to closed

Closing tickets for a deprecated plugin.

### Modify Ticket

Change Properties