Module doesn't use password_inc variable which prevents option of overwriting default password.inc file.

CommentFileSizeAuthor
#4 services-use_password_inc_variable-2622896-2.patch633 bytesstanislav mixnovich
#3 services-use_password_inc_variable-2622896.patch547 bytesstanislav mixnovich

Comments

boyan.borisov created an issue. See original summary.

tyler.frankenstein’s picture

tyler.frankenstein commented
Category: Bug report » Feature request
Status: Active » Postponed (maintainer needs more info)

@boyan.borisov, will you please share the file(s)/function(s)/line-numbers involved in this?

stanislav mixnovich’s picture

stanislav mixnovich commented
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new services-use_password_inc_variable-2622896.patch547 bytes

From the includes/password.inc file's comments:

* An alternative or custom version of this password hashing API may be
* used by setting the variable password_inc to the name of the PHP file
* containing replacement user_hash_password(), user_check_password(), and
* user_needs_new_hash() functions.

stanislav mixnovich’s picture

stanislav mixnovich commented
StatusFileSize
new services-use_password_inc_variable-2622896-2.patch633 bytes
tyler.frankenstein’s picture

tyler.frankenstein commented

Looks good. I'll wait for the test request to come back, and if all goes well we'll get this merged in, thanks!

marcingy’s picture

marcingy commented
Status: Needs review » Fixed

Merged thanks :)

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.