Adding error msg to ldap

@saurabh6790 The user bind and authentication methods doesn’t provide detailed error description in case of errors. Adding error_message to the exception events will make it easier to debug while working with ldap server issues. The code change is as simple as adding error_message to the exception event.

For eg changing
except ldap.LDAPError: to
except ldap.LDAPError, error_message:

and printing error_message is enough to get the error details from ldap.

Can this be added to the module code? I can create a pull request if needed.

1 Like

@kirthi Cool :slight_smile:

Sure please send PR.

FYI, with version 7.1 LDAP login is part of Frappe itself. We deprecated LDAP application.