diff options
author | Øyvind Grønnesby <oyving@yahooinc.com> | 2022-09-09 10:37:23 +0200 |
---|---|---|
committer | Øyvind Grønnesby <oyving@yahooinc.com> | 2022-09-09 10:37:23 +0200 |
commit | 537a6c824d72420c39aed93b1128d6ea9a8e6461 (patch) | |
tree | 749e2af2fe2ce6b1616d53fcfd95e449f1eeec9e | |
parent | 8ba6a5a53a5ecf0bee63c8a9430cca90e41b9909 (diff) |
Return empty list when user not found
-rw-r--r-- | controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/user/UserApiHandler.java | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/user/UserApiHandler.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/user/UserApiHandler.java index dddc3a47d69..893befb57a2 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/user/UserApiHandler.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/user/UserApiHandler.java @@ -145,21 +145,22 @@ public class UserApiHandler extends ThreadedHttpRequestHandler { private HttpResponse userMetadataFromUserId(String email) { var maybeUser = users.findUser(email); + var slime = new Slime(); + var root = slime.setObject(); + var usersRoot = root.setArray("users"); + if (maybeUser.isPresent()) { var user = maybeUser.get(); var roles = users.listRoles(new UserId(user.email())); - var slime = new Slime(); - var root = slime.setObject(); - var usersRoot = root.setArray("users"); renderUserMetaData(usersRoot.addObject(), user, Set.copyOf(roles)); - return new SlimeJsonResponse(slime); } - return ErrorResponse.notFoundError("Could not find user: " + email); + return new SlimeJsonResponse(slime); } private HttpResponse userMetadataQuery(String query) { var userList = users.findUsers(query); + var slime = new Slime(); var root = slime.setObject(); var userSlime = root.setArray("users"); |