Skip to content
This repository was archived by the owner on Jul 24, 2023. It is now read-only.

Hash::check changed in 5.7 #586

Closed
m-lotze opened this issue Sep 11, 2018 · 2 comments
Closed

Hash::check changed in 5.7 #586

m-lotze opened this issue Sep 11, 2018 · 2 comments

Comments

@m-lotze
Copy link

m-lotze commented Sep 11, 2018

  • Laravel Version: 5.7
  • Adldap2-Laravel Version: 4.0.7
  • PHP Version: 7.1
  • LDAP Type: ActiveDirectory

See this issue on laravel/framework:
laravel/framework#25578

If the ldap user is not in the database SyncPassword stops with a RuntimeException:
This password does not use the Bcrypt algorithm.

If this is not changed in laravel, you probably need to change line 88 at SyncPassword::passwordNeedsUpdate from:

if ($this->canSync()) {

to:
if ($current !== null && $this->canSync()) {

@stevebauman
Copy link
Member

Thanks for letting me know! Will be patched shortly :)

@stevebauman
Copy link
Member

Patched in v4.0.9.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants