Merge branch 'main' into 909-ldap-user-group-confirmation

Author: kriswest

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@finos/git-proxy",
-  "version": "1.17.1",
+  "version": "1.17.2",
   "description": "Deploy custom push protections and policies on top of Git.",
   "scripts": {
     "cli": "node ./packages/git-proxy-cli/index.js",

package.json

@@ -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;