search

DirectoryTree / LdapRecord-Laravel

Multi-domain LDAP Authentication & Management for Laravel.
https://ldaprecord.com/docs/laravel/v3
MIT License
483 stars 51 forks source link

[Bug] Must implement Illuminate\Contracts\Auth\UserProvider::rehashPasswordIfRequired method in Laravel 11 #638

Closed Grimthorr closed 4 months ago

Grimthorr commented 4 months ago

Environment:

Describe the bug: The following error appears in Laravel 11:

Class LdapRecord\Laravel\Auth\DatabaseUserProvider contains 1 abstract method and must therefore be declared abstract or implement the remaining methods (Illuminate\Contracts\Auth\UserProvider::rehashPasswordIfRequired)

From https://laravel.com/docs/11.x/upgrade#the-user-provider-contract:

The Illuminate\Contracts\Auth\UserProvider contract has received a new rehashPasswordIfRequired method.

This means that LdapRecord\Laravel\Auth\UserProvider should now implement a rehashPasswordIfRequired method for Laravel 11.

BucsaEmanuel commented 4 months ago

Same thing happened to me when I tried running php artisan ldap:import users on a fresh new Laravel 11 install with the package added and configured. Works in Laravel 10.

stevebauman commented 4 months ago

Thanks for the report! I've just released v3.3.1 with a patch including this new method.