-
-
Notifications
You must be signed in to change notification settings - Fork 4.7k
Use query builder instead of OC_DB in user_ldap #23933
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
Conversation
Signed-off-by: Joas Schilling <[email protected]>
Signed-off-by: Joas Schilling <[email protected]>
blizzz
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there's an array to string conversion issue on deleting configs
Signed-off-by: Arthur Schiwon <[email protected]>
Signed-off-by: Arthur Schiwon <[email protected]>
|
Mind to add the DB group to the tests in |
|
Lots of ldap issues in the NODB tests |
Signed-off-by: Joas Schilling <[email protected]>
|
🤖 beep boop beep 🤖 Here are the logs for the failed build: Status of 35235: failureintegration-comments-search
Show full logacceptance-users
Show full log |
MorrisJobke
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested LDAP wizard and provisioning and all still works 👍
Ref #23929