diff options
author | jonmv <venstad@gmail.com> | 2022-07-01 13:53:49 +0200 |
---|---|---|
committer | jonmv <venstad@gmail.com> | 2022-07-01 13:53:49 +0200 |
commit | 1f1218a0e5135f040cdf4360c938eeeae805442a (patch) | |
tree | f821a0dcb6b85245718d65a103b6d66b5810e1f3 /controller-server | |
parent | e4265f852d20675d873489973b3745580d2b3c5b (diff) |
Show tenants where user is hostedDeveloper
Diffstat (limited to 'controller-server')
-rw-r--r-- | controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/user/UserApiHandler.java | 18 |
1 files changed, 6 insertions, 12 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 fce2d283da2..e10defb4416 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 @@ -384,18 +384,12 @@ public class UserApiHandler extends ThreadedHttpRequestHandler { } private static Collection<TenantRole> filterTenantRoles(Role role) { - if (!(role instanceof TenantRole)) - return Set.of(); - - TenantRole tenantRole = (TenantRole) role; - if (tenantRole.definition() == RoleDefinition.administrator - || tenantRole.definition() == RoleDefinition.developer - || tenantRole.definition() == RoleDefinition.reader) - return Set.of(tenantRole); - - if (tenantRole.definition() == RoleDefinition.athenzTenantAdmin) - return Roles.tenantRoles(tenantRole.tenant()); - + if (role instanceof TenantRole tenantRole) { + switch (tenantRole.definition()) { + case administrator, developer, reader, hostedDeveloper: return Set.of(tenantRole); + case athenzTenantAdmin: return Roles.tenantRoles(tenantRole.tenant()); + } + } return Set.of(); } |