Merge pull request #928 from gentlementlegen/fix/crash-on-missing-mem…

…bership

fix: crash on missing membership

src/helpers/issue.ts

@@ -163,15 +163,19 @@ export async function isUserAdminOrBillingManager(
 
   async function checkIfIsBillingManager() {
     if (!payload.organization) throw context.logger.fatal(`No organization found in payload!`);
-    const { data: membership } = await context.octokit.rest.orgs.getMembershipForUser({
-      org: payload.organization.login,
-      username: payload.repository.owner.login,
-    });
-
-    console.trace(membership);
-    if (membership.role === "billing_manager") {
-      return true;
-    } else {
+    try {
+      const { data: membership } = await context.octokit.rest.orgs.getMembershipForUser({
+        org: payload.organization.login,
+        username: payload.repository.owner.login,
+      });
+
+      console.trace(membership);
+      return membership.role === "billing_manager";
+    } catch (e) {
+      context.logger.error(
+        `Could not get the Billing Manager status for ${payload.repository.owner.login} within ${payload.organization.login}`,
+        e
+      );
       return false;
     }
   }