switcht to member group id

broadinstitute · Oct 4, 2024 · d9c6050 · d9c6050
1 parent 4c3f5fa
commit d9c6050
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/...main/scala/org/broadinstitute/dsde/workbench/sam/dataAccess/PostgresAccessPolicyDAO.scala b/...main/scala/org/broadinstitute/dsde/workbench/sam/dataAccess/PostgresAccessPolicyDAO.scala
@@ -986,7 +986,7 @@ class PostgresAccessPolicyDAO(
         val deleteQuery = samsql"""
       delete from ${GroupMemberTable as gm}
       using ${PolicyTable as p}
-      where ${gm.groupId} = ${p.groupId}
+      where ${gm.memberGroupId} = ${p.groupId}
       and ${p.resourceId} = (${loadResourcePKSubQuery(resourceId)})
     """
         logger.info(s"deleteQuery: ${deleteQuery.statement}")

diff --git a/src/main/scala/org/broadinstitute/dsde/workbench/sam/service/ResourceService.scala b/src/main/scala/org/broadinstitute/dsde/workbench/sam/service/ResourceService.scala
@@ -497,6 +497,7 @@ class ResourceService(
     for {
       policiesToDelete <- accessPolicyDAO.listAccessPolicies(resource, samRequestContext)
       _ <- policiesToDelete.traverse { policy =>
+        logger.info(s"deleting policy ${policy}")
         cloudExtensions.onGroupDelete(policy.email)
       }
     } yield policiesToDelete