Skip to content
Snippets Groups Projects
Commit 9e63a098 authored by Daniel Gerhardt's avatar Daniel Gerhardt
Browse files

Do not manually split config parameter

parent 4aa6d35c
Branches
Tags
1 merge request!48Code cleanup
Pipeline #7917 passed with stages
in 3 minutes and 54 seconds
...@@ -42,7 +42,7 @@ import java.util.Arrays; ...@@ -42,7 +42,7 @@ import java.util.Arrays;
public class ApplicationPermissionEvaluator implements PermissionEvaluator { public class ApplicationPermissionEvaluator implements PermissionEvaluator {
@Value("${security.admin-accounts}") @Value("${security.admin-accounts}")
private String adminAccounts; private String[] adminAccounts;
@Autowired @Autowired
private IDatabaseDao dao; private IDatabaseDao dao;
...@@ -95,9 +95,7 @@ public class ApplicationPermissionEvaluator implements PermissionEvaluator { ...@@ -95,9 +95,7 @@ public class ApplicationPermissionEvaluator implements PermissionEvaluator {
private boolean checkAdminPermission(final String username) { private boolean checkAdminPermission(final String username) {
/* TODO: only allow accounts from arsnova db */ /* TODO: only allow accounts from arsnova db */
String[] splittedAdminNames = adminAccounts.split(","); return Arrays.asList(adminAccounts).contains(username);
return Arrays.asList(splittedAdminNames).contains(username);
} }
private boolean checkSessionPermission( private boolean checkSessionPermission(
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment