Steps to reproduce

go to /admin/reports/status >> The website has encountered an unexpected error.

/reports/dblog message :
TypeError : Cannot assign Drupal\mail_login\AuthDecorator to property Drupal\security_review\Plugin\SecurityCheck\NamePasswords::$userAuth of type Drupal\user\UserAuthInterface dans Drupal\security_review\Plugin\SecurityCheck\NamePasswords::create() (ligne 67 de /var/www/html/web/modules/contrib/security_review/src/Plugin/SecurityCheck/NamePasswords.php).

Comments

steveoriol created an issue. See original summary.

ananya.k’s picture

Status: Active » Needs review
StatusFileSize
new1.97 KB

@steveoriol,

The issues have been successfully resolved. Please refer to the attached patch for details.

steveoriol’s picture

Hi @ananya.k the patch "Error_in_mail_login_module-3462452.patch" does not apply
Oupss..., the patch is for the modue "mail_login"
and it works, no more errors, Thanks

steveoriol’s picture

Status: Needs review » Reviewed & tested by the community
greggles’s picture

Project: Security Review » Mail Login
Version: 3.0.2 » 4.0.x-dev

Should this be filed against the security review module or the mail_login module?

I think it's a patch for mail_login so I'm moving the project.

klemendev’s picture

Does this also happen in 4.0.2?

drupalbubb’s picture

Isn't the patch a downgrade from UserAuthenticationInterface to UserAuthInterface? My coding skills are deprecated, but looks like. If that is right, than this is no bug fix, just a quick fix. Shouldn't be a RTBC.
@KlemenDEV: this also happens in 4.0.2

klemendev’s picture

Status: Reviewed & tested by the community » Needs work
drupalbubb’s picture

zaporylie’s picture

Status: Needs work » Closed (duplicate)

Mail Login 4.0.3 was just released and supports the old UserAuthInterface and the new UserAuthenticationInterface. See #3464301: AuthDecorator must implement both the old UserAuthInterface and the new UserAuthenticationInterface