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

Activation of plugin bombs #19

Open
mladegaa opened this issue Feb 1, 2018 · 2 comments
Open

Activation of plugin bombs #19

mladegaa opened this issue Feb 1, 2018 · 2 comments

Comments

@mladegaa
Copy link

mladegaa commented Feb 1, 2018

While trying to activate the plugin, I get the following error message

Fatal error: Cannot use isset() on the result of a function call (you can use "null !== func()" instead) in /home/www/public_html/user/plugins/ldap-auth/plugin.php on line 283

should have the latest yourls and ldap plugin

@k3a
Copy link
Collaborator

k3a commented Feb 1, 2018

That should be fixed now, can you update the plugin to the master, test and report back please?

@mladegaa
Copy link
Author

mladegaa commented Feb 2, 2018

The fix resolved the first issue.

The plugin activates now, but now i'm getting these errors

Warning: array_merge(): Argument #2 is not an array in /home/www/public_html/user/plugins/ldap-auth/plugin.php on line 285

Warning: ldap_search(): Search: Operations error in /home/www/public_html/user/plugins/ldap-auth/plugin.php on line 189

And neither LDAP or local users work, can log in with local users if I rename the plugin file.

Not sure if our LDAP allows anonymous search, does the search happen before or after the bind with the submitted user credentials?

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

No branches or pull requests

2 participants