Fixed Issue #7. Other solution for pull request #8 - it is now selectable which admin interface is used (via .env file)

2021-05-10 23:40:49 +02:00
parent 90e7681c35
commit 57da632558
6 changed files with 91 additions and 4 deletions
--- a/ldap-overleaf-sl/sharelatex/AuthenticationManager.js
+++ b/ldap-overleaf-sl/sharelatex/AuthenticationManager.js
@@ -306,8 +306,8 @@ const AuthenticationManager = {
      }
    } catch (ex) {
      console.log("An Error occured while getting user data during ldapsearch: " + String(ex))
-        await client.unbind();
-        return callback(null, null)
+      await client.unbind();
+      return callback(null, null)
    }
    try {
      // if admin filter is set - only set admin for user in ldap group