Skip to content
This repository has been archived by the owner on Dec 13, 2022. It is now read-only.

fix(ldap): add debug and avoid ldap failures #11827

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from
Open

Conversation

kduret
Copy link
Contributor

@kduret kduret commented Sep 21, 2022

Description

add debug and avoid ldap failures

Handle following error :
[15-Sep-2022 10:31:20 GMT] CRITICAL: Uncaught Error: ldap_get_entries(): Argument #2 ($result) must be of type resource, bool given {"exception":"[object] (TypeError(code: 0): ldap_get_entries(): Argument #2 ($result) must be of type resource, bool given at /usr/share/centreon/www/class/centreonLDAP.class.php:326)"}

Add debug to avoid to be blind when errors happen

Fixes MON-15115

Type of change

  • Patch fixing an issue (non-breaking change)
  • New functionality (non-breaking change)
  • Breaking change (patch or feature) that might cause side effects breaking part of the Software

Target serie

  • 21.04.x
  • 21.10.x
  • 22.04.x
  • 22.10.x (master)

@kduret kduret marked this pull request as ready for review September 21, 2022 15:13
@kduret kduret requested review from a team, adr-mo and a-launois September 21, 2022 15:13
@sonarqube-decoration
Copy link

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

Successfully merging this pull request may close these issues.

5 participants