Skip to content

Commit c6c7b51

Browse files
use ReservedRoleNameChecker.isReserved consistently
1 parent 9a21687 commit c6c7b51

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/action/role/TransportGetRolesAction.java

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -66,13 +66,10 @@ protected void doExecute(Task task, final GetRolesRequest request, final ActionL
6666
final Set<RoleDescriptor> reservedRoles = new LinkedHashSet<>();
6767
if (specificRolesRequested) {
6868
for (String role : requestedRoles) {
69-
if (ReservedRolesStore.isReserved(role)) {
69+
if (reservedRoleNameChecker.isReserved(role)) {
7070
RoleDescriptor rd = ReservedRolesStore.roleDescriptor(role);
7171
if (rd != null) {
7272
reservedRoles.add(rd);
73-
} else {
74-
listener.onFailure(new IllegalStateException("unable to obtain reserved role [" + role + "]"));
75-
return;
7673
}
7774
} else {
7875
rolesToSearchFor.add(role);

0 commit comments

Comments
 (0)