Replies: 4 comments 7 replies
-
Hi @ale1981, Could you elaborate on Laravel Nova support? What are you looking to achieve with Laravel Nova & LdapRecord? Authentication? LDAP management? Etc. |
Beta Was this translation helpful? Give feedback.
-
@stevebauman I'd like to chime into this discussion. We are running against the same issues with Laravel Nova. Nova is resolving the user model via the following code:
When the Since the So the set-up as described in the "Database Auth Configuration" of the LdapRecord library unfortunately breaks compatibility with Laravel Nova. Maybe it's possible to turn the config around and make the Eloquent user the default overwriting the Config with an "ldap" array? Just a thought.
If you need any more input, please let me know. I'd love to help resolving this. |
Beta Was this translation helpful? Give feedback.
-
@stevebauman Thanks for the quick reply and the link to the issue with more details. I have read through it and agree somewhat that it would probably be better/easier to solve this within the LdapRecord library instead of having to force other libraries that follow the Laravel convention to implement the resolver. This way it also won't break future third-party packages that follow the convention. I also understand that you don't want to and cannot introduce breaking changes within your library. So I am trying to come up with some non-breaking ideas. How about the following approach: Plain Auth
Database Auth I am not too familiar with the ins and outs of the LdapRecord library. But since the 'database' key must be read by the library, so should an 'ldap' key be readable. So wouldn't it be possible to support both configurations:
as well as
whereas the LdapRecord library can configure the internals depending on whether a "database" key or an "ldap" key is present. This could solve the issues with third-party packages that rely on the eloquent user model configuration. Would that be possible? |
Beta Was this translation helpful? Give feedback.
-
@stevebauman Thanks for your feedback! I have come up with an idea, please see #639 |
Beta Was this translation helpful? Give feedback.
-
Hi,
Before I contribute / sponsor and raise an Issue is Laravel Nova supported?
Thanks
Beta Was this translation helpful? Give feedback.
All reactions