summary refs log tree commit diff
path: root/synapse/handlers
diff options
context:
space:
mode:
authorErik Johnston <erik@matrix.org>2016-04-15 11:17:18 +0100
committerErik Johnston <erik@matrix.org>2016-04-15 11:17:18 +0100
commit6fd2f685fe5722326a7719892368d0fa2aa92efa (patch)
treec34d9aefc2d9fa020588b1bd7dd6cc13d8a99def /synapse/handlers
parentMerge pull request #731 from matrix-org/erikj/timed_otu (diff)
downloadsynapse-6fd2f685fe5722326a7719892368d0fa2aa92efa.tar.xz
Simplify _check_password
Diffstat (limited to 'synapse/handlers')
-rw-r--r--synapse/handlers/auth.py14
1 files changed, 9 insertions, 5 deletions
diff --git a/synapse/handlers/auth.py b/synapse/handlers/auth.py
index 916632c7d7..61fe56032a 100644
--- a/synapse/handlers/auth.py
+++ b/synapse/handlers/auth.py
@@ -432,11 +432,15 @@ class AuthHandler(BaseHandler):
         Returns:
             True if the user_id successfully authenticated
         """
-        defer.returnValue((
-            (yield self._check_ldap_password(user_id, password))
-            or
-            (yield self._check_local_password(user_id, password))
-        ))
+        valid_ldap = yield self._check_ldap_password(user_id, password)
+        if valid_ldap:
+            defer.returnValue(True)
+
+        valid_local_password = yield self._check_local_password(user_id, password)
+        if valid_local_password:
+            defer.returnValue(True)
+
+        defer.returnValue(False)
 
     @defer.inlineCallbacks
     def _check_local_password(self, user_id, password):