fix: 🐛 remove all-org-members which is pushing over char limit

ministryofjustice · Oct 24, 2024 · 8813f78 · 8813f78
1 parent d96cab7
commit 8813f78
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/add-github-teams-to-aws-saml.js b/add-github-teams-to-aws-saml.js
@@ -10,7 +10,9 @@ exports.onExecutePostLogin = async (event, api) => {
       t.replace("github:", ""),
     );
 
-    api.user.GithubTeam = ":" + git_teams.join(":") + ":";
+    const filteredTeams = git_teams.filter((t) => t != "all-org-members");
+
+    api.user.GithubTeam = ":" + filteredTeams.join(":") + ":";
     api.user.awsRoleSession = event.user.nickname;
     api.user.awsTagKeys = ["GithubTeam"];
     api.user.awsRole = rolePrefix + ":role/" + role + "," + samlIdP;
@@ -27,7 +29,7 @@ exports.onExecutePostLogin = async (event, api) => {
 
     api.samlResponse.setAttribute(
       "https://aws.amazon.com/SAML/Attributes/PrincipalTag:GithubTeam",
-      ":" + git_teams.join(":") + ":",
+      ":" + filteredTeams.join(":") + ":",
     );
   }
 };