address comments

Author: Bencheng21

pkg/connector/connector.go

@@ -451,7 +451,7 @@ func getOrgs(ctx context.Context, client *github.Client, orgs []string) ([]strin
 		orgs, resp, err := client.Organizations.List(ctx, "", &github.ListOptions{Page: page, PerPage: maxPageSize})
 		if err != nil {
 			if isRatelimited(resp) {
-				return nil, nil
+				return nil, uhttp.WrapErrors(codes.Unavailable, "too many requests", err)
 			}
 			return nil, fmt.Errorf("github-connector: failed to retrieve org: %w", err)
 		}

pkg/connector/org.go

@@ -131,7 +131,7 @@ func (o *orgResourceType) List(
 			}
 
 			if isRatelimited(resp) {
-				return ret, "", nil, uhttp.WrapErrors(codes.Unavailable, "too many requests", err)
+				return nil, "", nil, uhttp.WrapErrors(codes.Unavailable, "too many requests", err)
 			}
 			return nil, "", nil, err
 		}

pkg/connector/org_role.go

@@ -91,7 +91,7 @@ func (o *orgRoleResourceType) List(
 			return nil, "", nil, nil
 		}
 		if isRatelimited(resp) {
-			uhttp.WrapErrors(codes.Unavailable, "too many requests", err)
+			return nil, "", nil, uhttp.WrapErrors(codes.Unavailable, "too many requests", err)
 		}
 		return nil, "", nil, fmt.Errorf("failed to list organization roles: %w", err)
 	}

pkg/connector/user.go

@@ -163,7 +163,7 @@ func (o *userResourceType) List(ctx context.Context, parentID *v2.ResourceId, pt
 			if res == nil || res.StatusCode != http.StatusNotFound {
 				return nil, "", nil, err
 			}
-			if isRatelimited(resp) {
+			if isRatelimited(res) {
 				return nil, "", nil, uhttp.WrapErrors(codes.Unavailable, "too many requests", err)
 			}
 			l.Error("error fetching user by id", zap.Error(err), zap.Int64("user_id", user.GetID()))