[Bug] v3 - Argument #1 ($guid) must be of type string, null given

DirectoryTree / LdapRecord-Laravel

Multi-domain LDAP Authentication & Management for Laravel.

https://ldaprecord.com/docs/laravel/v3

MIT License

492 stars 51 forks source link

[Bug] v3 - Argument #1 ($guid) must be of type string, null given #538

Closed m-lotze closed 1 year ago

m-lotze commented 1 year ago

We have a mixture of Ldap users and local eloquent users. After updating to version 3 we receive the exception: Argument #1 ($guid) must be of type string, null given in LdapRecord\Laravel\LdapUserRepository->findByGuid() when trying to authenticate a local eloquent user where the guid is null in our database. This worked in version 2. Is it required to generate a guid for local users in version 3?

stevebauman commented 1 year ago

Thanks for the report @m-lotze! This is a bug -- fixing now 🙏

stevebauman commented 1 year ago

Hi @m-lotze, I've just released a patch for this issue (v3.0.1). Run composer update and you should be all set! 👍