fix(anonymous user): Only put anonymous user in Redis DB if it's not …

…already present (#193)
spinnaker · Jul 11, 2017 · c97f935 · c97f935
1 parent 616801b
commit c97f935
Showing 1 changed file with 3 additions and 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
@@ -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;
   }
 }