Merge pull request #148 from hanix/master

added pagination to Roles
SSHcom · May 22, 2024 · d36e7c2 · d36e7c2
2 parents 4208b8c + 758f557
commit d36e7c2
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/api/rolestore/client.go b/api/rolestore/client.go
@@ -183,12 +183,18 @@ func (store *RoleStore) LinkedRoles(awsroleID string) ([]AWSRoleLink, error) {
 }
 
 // Roles gets all configured roles.
-func (store *RoleStore) Roles() ([]Role, error) {
+func (store *RoleStore) Roles(offset, limit int, sortkey, sortdir string) ([]Role, error) {
 	result := rolesResult{}
 
+	filters := Params{
+		Offset:  offset,
+		Limit:   limit,
+		Sortkey: sortkey,
+		Sortdir: sortdir,
+	}
+
 	_, err := store.api.
-		URL("/role-store/api/v1/roles").
-		Get(&result)
+		URL("/role-store/api/v1/roles").Query(filters).Get(&result)
 
 	return result.Items, err
 }