Merge pull request #24 from appuio/fix-remove-users-pagination

Remove users pagination by setting max to `-1`, correctly pass parameter
appuio · Feb 28, 2022 · 402033c · 402033c
2 parents 42495c6 + eabe398
commit 402033c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/internal/pkg/keycloak/client.go b/internal/pkg/keycloak/client.go
@@ -38,5 +38,5 @@ func (g *gocloakClient) GetUsers(ctx context.Context, realm string, params goclo
 		return nil, fmt.Errorf("failed binding to keycloak: %w", err)
 	}
 
-	return g.client.GetUsers(ctx, token.AccessToken, realm, gocloak.GetUsersParams{})
+	return g.client.GetUsers(ctx, token.AccessToken, realm, params)
 }
diff --git a/internal/pkg/sync/sync.go b/internal/pkg/sync/sync.go
@@ -23,7 +23,7 @@ type UserSyncer struct {
 
 func (u *UserSyncer) Sync(ctx context.Context, realm, attribute, targetLabel, targetAnnotation string) error {
 	users, err := u.KeycloakClient.GetUsers(ctx, realm, gocloak.GetUsersParams{
-		Max: gocloak.IntP(5000),
+		Max: gocloak.IntP(-1),
 	})
 	if err != nil {
 		return fmt.Errorf("error fetching users: %w", err)