summaryrefslogtreecommitdiffstats
path: root/controller-server
diff options
context:
space:
mode:
authorjonmv <venstad@gmail.com>2022-07-01 13:53:49 +0200
committerjonmv <venstad@gmail.com>2022-07-01 13:53:49 +0200
commit1f1218a0e5135f040cdf4360c938eeeae805442a (patch)
treef821a0dcb6b85245718d65a103b6d66b5810e1f3 /controller-server
parente4265f852d20675d873489973b3745580d2b3c5b (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.java18
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();
}