Skip to content

Commit 2908f13

Browse files
committed
fix: ensure role members endpoint only returns the ids instead of objects
1 parent 73a4f34 commit 2908f13

File tree

4 files changed

+4
-3
lines changed

4 files changed

+4
-3
lines changed

src/main/java/com/hawk/keycloak/ApiRoot.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -158,7 +158,7 @@ public Stream<RoleRepresentation> getRoles(
158158
@GET
159159
@Path("roles/{role}/members")
160160
@Produces(MediaType.APPLICATION_JSON)
161-
public Stream<Map<String, String>> getRoleMembers(
161+
public Stream<String> getRoleMembers(
162162
@Parameter(description = "The id of the role to find the members of") @PathParam("role") String roleId,
163163
@Parameter(description = "Pagination offset") @QueryParam("first") Integer firstResult,
164164
@Parameter(description = "Maximum results size (defaults to 100)") @QueryParam("max") Integer maxResults

src/main/java/com/hawk/keycloak/roles/RolesRequestHandler.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
import org.keycloak.models.KeycloakSession;
99
import org.keycloak.models.RealmModel;
1010
import org.keycloak.models.RoleModel;
11+
import org.keycloak.models.UserModel;
1112
import org.keycloak.models.utils.ModelToRepresentation;
1213
import org.keycloak.representations.idm.RoleRepresentation;
1314

@@ -47,7 +48,7 @@ public Stream<RoleRepresentation> handleRolesRequest(
4748

4849
}
4950

50-
public Stream<Map<String, String>> handleRoleMembersRequest(
51+
public Stream<String> handleRoleMembersRequest(
5152
String roleId,
5253
Integer firstResult,
5354
Integer maxResults
@@ -70,6 +71,6 @@ public Stream<Map<String, String>> handleRoleMembersRequest(
7071
}
7172

7273
return session.users().getRoleMembersStream(realm, roleModel, firstResult, maxResults)
73-
.map(m -> Map.of("id", m.getId()));
74+
.map(UserModel::getId);
7475
}
7576
}
-19 Bytes
Binary file not shown.
-101 Bytes
Binary file not shown.

0 commit comments

Comments
 (0)