Merge pull request #60 from ConductorOne/BB614

[BB-614] baton-github: ignore not found error

Author: Bencheng21

pkg/connector/helpers.go

@@ -3,6 +3,7 @@ package connector
 import (
 	"context"
 	"fmt"
+	"net/http"
 	"strconv"
 	"strings"
 	"sync"
@@ -212,3 +213,10 @@ type hasSAMLQuery struct {
 		}
 	} `graphql:"organization(login: $orgLoginName)"`
 }
+
+func isNotFoundError(resp *github.Response) bool {
+	if resp == nil {
+		return false
+	}
+	return resp.StatusCode == http.StatusNotFound
+}

pkg/connector/team.go

@@ -12,9 +12,11 @@ import (
 	"github.com/conductorone/baton-sdk/pkg/types/entitlement"
 	"github.com/conductorone/baton-sdk/pkg/types/grant"
 	rType "github.com/conductorone/baton-sdk/pkg/types/resource"
+	"github.com/conductorone/baton-sdk/pkg/uhttp"
 	"github.com/google/go-github/v63/github"
 	"github.com/grpc-ecosystem/go-grpc-middleware/logging/zap/ctxzap"
 	"go.uber.org/zap"
+	"google.golang.org/grpc/codes"
 )
 
 const (
@@ -102,8 +104,11 @@ func (o *teamResourceType) List(ctx context.Context, parentID *v2.ResourceId, pt
 	}
 
 	for _, team := range teams {
-		fullTeam, _, err := o.client.Teams.GetTeamByID(ctx, orgID, team.GetID())
+		fullTeam, resp, err := o.client.Teams.GetTeamByID(ctx, orgID, team.GetID())
 		if err != nil {
+			if isNotFoundError(resp) {
+				return nil, "", nil, uhttp.WrapErrors(codes.NotFound, fmt.Sprintf("team: %d not found", team.GetID()))
+			}
 			return nil, "", nil, err
 		}
 
@@ -178,6 +183,9 @@ func (o *teamResourceType) Grants(ctx context.Context, resource *v2.Resource, pT
 
 	users, resp, err := o.client.Teams.ListTeamMembersByID(ctx, org.GetID(), githubID, &opts)
 	if err != nil {
+		if isNotFoundError(resp) {
+			return nil, "", nil, uhttp.WrapErrors(codes.NotFound, fmt.Sprintf("org: %d not found", org.GetID()))
+		}
 		return nil, "", nil, fmt.Errorf("github-connectorv2: failed to fetch team members: %w", err)
 	}
 
@@ -195,6 +203,9 @@ func (o *teamResourceType) Grants(ctx context.Context, resource *v2.Resource, pT
 	for _, user := range users {
 		membership, _, err := o.client.Teams.GetTeamMembershipByID(ctx, org.GetID(), githubID, user.GetLogin())
 		if err != nil {
+			if isNotFoundError(resp) {
+				return nil, "", nil, uhttp.WrapErrors(codes.NotFound, fmt.Sprintf("user: %s not found", user.GetLogin()))
+			}
 			return nil, "", nil, fmt.Errorf("github-connectorv2: failed to get team membership for user: %w", err)
 		}