Project

General

Profile

Actions

Todo #13718

closed

Improve LDAP debugging

Added by Jim Pingle almost 2 years ago. Updated almost 2 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Authentication
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
23.01
Release Notes:
Default

Description

The LDAP debug logs are inconsistent in their use of log_auth() vs log_error() and they should all be log_error() as using log_auth() will result in console alerts.

Also the messages could use a consistent prefix such as "LDAP Debug".

Would also help to log debug info in a few more cases along the code path plus a summary of settings.

And then to make it actually useful, add a 'debug' option to diag_authentication.php which will trigger this logging.

See also: #13093

To me, I already have the changes done, this is to make sure it gets listed in the release notes.

N.B.: There is still room for further improvement in future versions and also we could add similar debugging to radius auth and so on eventually.

Actions

Also available in: Atom PDF