Unset global password can cause Deprecated function error

Created on 6 September 2023, over 1 year ago

Updated 18 January 2024, 11 months ago

Problem/Motivation

If module is set to check for a global password but a global password is not set in configuration, the user may encounter:

Deprecated function: password_verify(): Passing null to parameter #2 ($hash) of type string is deprecated in Drupal\Core\Password\PhpPassword->check() (line 49 of core/lib/Drupal/Core/Password/PhpPassword.php).

Steps to reproduce

Install module
Set Global password setting to "Allow per page or Global" or "Allow only Global"
Do not set a Global Password
Add a protected page and attempt to authenticate

Proposed resolution

Check for null value on global password before check() call.

🐛 Bug report

Status

Needs review

Version

1.0

Component

Code

Created by

dkosbob

Live updates comments and jobs are added and updated live.

Merge Requests

!23Unset global password can cause Deprecated function error
Open
🇮🇳India abhishek_virasat
updated 11 months ago

Comments & Activities

Issue created by @dkosbob
Comment over 1 year ago →
dkosbob
Patch attached.
Comment 11 months ago →
🇮🇳India abhishek_virasat
abhishek_gupta1 → made their first commit to this issue’s fork.
Merge request !23fix the issue Unset global password can cause Deprecated function error → (Open) created by abhishek_virasat
Status changed to Needs review 11 months ago11:52am 18 January 2024
Comment 11 months ago →
🇮🇳India abhishek_virasat
@oksana-c, I have fixed the issue and created MR. please kindly review the mR

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024