Merge pull request #1203 from SciCatProject/bug-fix-empty-string-valu…

…e-in-accessGroups fix: empty string value in accessGroups field
SciCatProject · May 1, 2024 · 05965bc · 05965bc
2 parents c0ef6e2 + 857530a
commit 05965bc
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/auth/access-group-provider/access-group-from-multiple-providers.service.ts b/src/auth/access-group-provider/access-group-from-multiple-providers.service.ts
@@ -17,8 +17,12 @@ export class AccessGroupFromMultipleProvidersService extends AccessGroupService
     for (const accessGroupProvider of this.accessGroupProviders) {
       const accessGroupsFromProvider =
         await accessGroupProvider.getAccessGroups(userPayload);
-      accessGroups.push(...accessGroupsFromProvider);
+
+      accessGroups.push(
+        ...accessGroupsFromProvider.filter((group) => group.trim() !== ""),
+      );
     }
+
     return accessGroups;
   }
 }