Merge pull request #1046 from 06kellyjac/ad_fix

fix: correct method for finding ad configuration

Author: JamieSlome

src/service/passport/activeDirectory.js

@@ -1,15 +1,15 @@
 const ActiveDirectoryStrategy = require('passport-activedirectory');
 const ldaphelper = require('./ldaphelper');
 
-const type = "activedirectory";
+const type = 'activedirectory';
 
 const configure = (passport) => {
   const db = require('../../db');
 
   // We can refactor this by normalizing auth strategy config and pass it directly into the configure() function,
   // ideally when we convert this to TS.
   const authMethods = require('../../config').getAuthMethods();
-  const config = authMethods.find((method) => method.type.toLowerCase() === "activeDirectory");
+  const config = authMethods.find((method) => method.type.toLowerCase() === type);
   const adConfig = config.adConfig;
 
   const { userGroup, adminGroup, domain } = config;
@@ -59,7 +59,7 @@ const configure = (passport) => {
           console.log(`Error authenticating AD user: ${err.message}`);
           return done(err, null);
         }
-      }
+      },
     ),
   );