openrat-cms

commit a7148d721d1fb955adf81cac496ac2c214406465
parent 199daeddbb6c5224958377a13862a6131732a6a5
Author: Jan Dankert <develop@jandankert.de>
Date:   Mon, 30 Nov 2020 11:17:56 +0100

Fix: DatabaseAuth should only use the Auth::STATUS_* constants as return value.

Diffstat:
M
modules/cms/auth/DatabaseAuth.class.php
 | 
5
++---

1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/modules/cms/auth/DatabaseAuth.class.php b/modules/cms/auth/DatabaseAuth.class.php
@@ -2,7 +2,6 @@
 
 namespace cms\auth;
 
-use cms\auth\Auth;
 use cms\base\Configuration;
 use database\Database;
 
@@ -22,7 +21,7 @@ class DatabaseAuth implements Auth
 		$authDbConf = Configuration::subset(['security','authdb']);
 
 		if (!$authDbConf->is('enable',true))
-			return false;
+			return Auth::STATUS_FAILED;
 
 		$authdb = new Database($authDbConf);
 
@@ -32,7 +31,7 @@ class DatabaseAuth implements Auth
 		$sql->setString('password', hash($algo, $password));
 		$row = $sql->getRow();
 
-		// noch nicht implementiert: $authdb->close();
+		$authdb->disconnect();
 
 		return $row ? Auth::STATUS_SUCCESS : Auth::STATUS_FAILED;
 	}