[icinga-checkins] icinga.org: icingaweb2/feature/doc-search-6630: LdapUserBackend: disable " health check"

git at icinga.org git at icinga.org
Wed Feb 11 13:28:19 CET 2015

Module: icingaweb2
Branch: feature/doc-search-6630
Commit: 81f65a7cd4dadab8e31f4aa41092649551a8f20e
URL:    https://git.icinga.org/?p=icingaweb2.git;a=commit;h=81f65a7cd4dadab8e31f4aa41092649551a8f20e

Author: Thomas Gelf <thomas at gelf.net>
Date:   Mon Feb  9 15:29:52 2015 +0100

LdapUserBackend: disable "health check"

I see no point in checking this at every login. It could however be a
nice addition for our config backends and the setup wizard. I'd also
opt for completely removing this parameter - who wants to use this
method should explicitely call it.


 library/Icinga/Authentication/Backend/LdapUserBackend.php |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/library/Icinga/Authentication/Backend/LdapUserBackend.php b/library/Icinga/Authentication/Backend/LdapUserBackend.php
index 7fde1f6..016512a 100644
--- a/library/Icinga/Authentication/Backend/LdapUserBackend.php
+++ b/library/Icinga/Authentication/Backend/LdapUserBackend.php
@@ -170,7 +170,7 @@ class LdapUserBackend extends UserBackend
      * @throws  AuthenticationException     In case an error occured or the health check has failed
-    public function authenticate(User $user, $password, $healthCheck = true)
+    public function authenticate(User $user, $password, $healthCheck = false)
         if ($healthCheck) {
             try {

More information about the icinga-checkins mailing list