Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
Grimthorr opened this issue Mar 13, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@Grimthorr
Copy link

Environment:

  • LdapRecord-Laravel Version: ^3.3
  • PHP Version: ^8.2
  • Laravel Version: ^11.0

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.

@Grimthorr Grimthorr added the bug Something isn't working label Mar 13, 2024
@BucsaEmanuel
Copy link

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
Copy link
Member

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants