From c97f93512f3b1decdfa80411cc667a994c132582 Mon Sep 17 00:00:00 2001 From: Travis Tomsu Date: Tue, 11 Jul 2017 14:40:50 -0400 Subject: [PATCH] fix(anonymous user): Only put anonymous user in Redis DB if it's not already present (#193) --- .../spinnaker/fiat/config/UnrestrictedResourceConfig.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/fiat-roles/src/main/java/com/netflix/spinnaker/fiat/config/UnrestrictedResourceConfig.java b/fiat-roles/src/main/java/com/netflix/spinnaker/fiat/config/UnrestrictedResourceConfig.java index e16a4d754..e9f6a12e0 100644 --- a/fiat-roles/src/main/java/com/netflix/spinnaker/fiat/config/UnrestrictedResourceConfig.java +++ b/fiat-roles/src/main/java/com/netflix/spinnaker/fiat/config/UnrestrictedResourceConfig.java @@ -31,7 +31,9 @@ public class UnrestrictedResourceConfig { @Bean @ConditionalOnExpression("${fiat.writeMode.enabled:true}") String addUnrestrictedUser(PermissionsRepository permissionsRepository) { - permissionsRepository.put(new UserPermission().setId(UNRESTRICTED_USERNAME)); + if (!permissionsRepository.get(UNRESTRICTED_USERNAME).isPresent()) { + permissionsRepository.put(new UserPermission().setId(UNRESTRICTED_USERNAME)); + } return UNRESTRICTED_USERNAME; } }