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.

Sign up for GitHub

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

Jump to bottom

Add debug options to LDAP for diagnostic purposes #4547

Closed
TheWitness opened this issue Feb 6, 2022 · 1 comment
Closed

Add debug options to LDAP for diagnostic purposes #4547

TheWitness opened this issue Feb 6, 2022 · 1 comment
Labels
bug Undesired behaviour resolved A fixed issue
Milestone
v1.2.20

Comments

@TheWitness
Copy link
Member

Describe the bug

When attempting to login with LDAP, backtrace errors are being generated.

AUTH LDAP: Protocol Error, unable to start TLS communications (Can't contact LDAP server) on Server (192.168.11.105)
ERROR PHP NOTICE: Undefined index: error_num in file: /var/www/html/cacti/lib/ldap.php  on line: 177
ERROR PHP NOTICE: Undefined index: error_num in file: /var/www/html/cacti/auth_login.php  on line: 160

Seems related to recent LDAP changes in error handling.

Expected behavior

LDAP, even with configuration issues should not generate backtraces.
@TheWitness TheWitness added the bug Undesired behaviour label Feb 6, 2022
@TheWitness TheWitness added this to the v1.2.20 milestone Feb 6, 2022
TheWitness added a commit that referenced this issue Feb 6, 2022
@TheWitness
Fixing #4547 - LDaP Connection Backtraces and add LDAP Debug mode
aae896c 
- Testing with LDAP authentication generating backtraces
- Add LDAP Debug Mode option for testing LDAP specific connection issues
@TheWitness TheWitness added the resolved A fixed issue label Feb 6, 2022
@TheWitness
Copy link
Member Author

Added 'debug' mode to LDAP. Fixed the error now.

TheWitness added a commit that referenced this issue Feb 6, 2022
@TheWitness
Missed one connection error and change order on debug for #4547
71f6583 
This changes things to be a bit more reliable and I missed one connection error handling issue.
@netniV netniV changed the title Testing with LDAP authentication generating backtraces Add debug options to LDAP for diagnostic purposes Apr 3, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Dec 2, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Undesired behaviour resolved A fixed issue
Projects
None yet
Milestone
v1.2.20
Development

No branches or pull requests
1 participant
@TheWitness