Skip to content

Commit 9261b6e

Browse files
pradeepagrawal8184mneethiraj
authored andcommitted
RANGER-3807: getUserRoles API gives 200 for non existing user passed to this API
(cherry picked from commit d247fec)
1 parent 617225d commit 9261b6e

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

security-admin/src/main/java/org/apache/ranger/rest/RoleREST.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -698,6 +698,9 @@ public List<String> getUserRoles(@PathParam("user") String userName, @Context Ht
698698
LOG.debug("==> getUserRoles()");
699699
}
700700
try {
701+
if (xUserService.getXUserByUserName(userName) == null) {
702+
throw restErrorUtil.createRESTException(HttpServletResponse.SC_NOT_FOUND, "User:" + userName + " not found", false);
703+
}
701704
Set<RangerRole> roleList = roleStore.getRoleNames(userName, userMgr.getGroupsForUser(userName));
702705
for (RangerRole role : roleList) {
703706
ret.add(role.getName());

0 commit comments

Comments
 (0)