Merge pull request #11 from ConductorOne/ggreer/page-access-levels

Fetch page access levels for a group once instead of re-fetching for every member of a group.

pkg/connector/pages.go

@@ -173,17 +173,17 @@ func (s *pageSyncer) Grants(ctx context.Context, resource *v2.Resource, pToken *
 			return nil, "", nil, err
 		}
 
+		pageAccessLevels, err := s.pageAccessLevelsForGroup(ctx, page, group)
+		if err != nil {
+			return nil, "", nil, err
+		}
+
 		for _, m := range members {
 			if m.GetUserID() == 0 {
 				l.Debug("member did not have user ID defined -- skipping")
 				continue
 			}
 
-			pageAccessLevels, err := s.pageAccessLevelsForGroup(ctx, page, group)
-			if err != nil {
-				return nil, "", nil, err
-			}
-
 			principalID := &v2.ResourceId{
 				ResourceType: resourceTypeUser.Id,
 				Resource:     formatObjectID(resourceTypeUser.Id, m.GetUserID()),