Avoid DEP0174 by removing async from callback-based getGroupPolicyForUser

services/web/modules/authentication/ldap/app/src/LDAPModuleManager.mjs

@@ -94,18 +94,18 @@ const LDAPModuleManager = {
       logger.info({}, error.message)
     }
   },
-  async getGroupPolicyForUser(user, callback) {
-    try {
-      const userValidationMap = await PermissionsManager.promises.getUserValidationStatus({
-        user,
-        groupPolicy : { 'ldapPolicy' : true },
-        subscription : null
-      })
+
+  getGroupPolicyForUser(user, callback) {
+    PermissionsManager.promises.getUserValidationStatus({
+      user,
+      groupPolicy : { 'ldapPolicy' : true },
+      subscription : null
+    }).then(userValidationMap => {
       let groupPolicy = Object.fromEntries(userValidationMap)
-      callback(null,  {'groupPolicy' : groupPolicy })
-    } catch (error) {
+      callback(null, { groupPolicy })
+    }).catch(error => {
       callback(error)
-    }
+    })
   },
 }
 

services/web/modules/authentication/oidc/app/src/OIDCModuleManager.mjs

@@ -64,18 +64,18 @@ const OIDCModuleManager = {
       logger.info({}, error.message)
     }
   },
-  async getGroupPolicyForUser(user, callback) {
-    try {
-      const userValidationMap = await PermissionsManager.promises.getUserValidationStatus({
-        user,
-        groupPolicy : { 'oidcPolicy' : true },
-        subscription : null
-      })
+
+  getGroupPolicyForUser(user, callback) {
+    PermissionsManager.promises.getUserValidationStatus({
+      user,
+      groupPolicy : { 'oidcPolicy' : true },
+      subscription : null
+    }).then(userValidationMap => {
       let groupPolicy = Object.fromEntries(userValidationMap)
-      callback(null,  {'groupPolicy' : groupPolicy })
-    } catch (error) {
+      callback(null, { groupPolicy })
+    }).catch(error => {
       callback(error)
-    }
+    })
   },
 }
 

services/web/modules/authentication/saml/app/src/SAMLModuleManager.mjs

@@ -84,18 +84,17 @@ const SAMLModuleManager = {
       logger.info({}, error.message)
     }
   },
-  async getGroupPolicyForUser(user, callback) {
-    try {
-      const userValidationMap = await PermissionsManager.promises.getUserValidationStatus({
-        user,
-        groupPolicy : { 'samlPolicy' : true },
-        subscription : null
-      })
+  getGroupPolicyForUser(user, callback) {
+    PermissionsManager.promises.getUserValidationStatus({
+      user,
+      groupPolicy : { 'samlPolicy' : true },
+      subscription : null
+    }).then(userValidationMap => {
       let groupPolicy = Object.fromEntries(userValidationMap)
-      callback(null,  {'groupPolicy' : groupPolicy })
-    } catch (error) {
+      callback(null, { groupPolicy })
+    }).catch(error => {
       callback(error)
-    }
+    })
   },
 }