Remove synchronized

Signed-off-by: Craig Perkins <[email protected]>
opensearch-project · Aug 7, 2023 · 30095c8 · 30095c8
1 parent c604f93
commit 30095c8
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/main/java/org/opensearch/security/user/User.java b/src/main/java/org/opensearch/security/user/User.java
@@ -272,13 +272,13 @@ public synchronized final Map<String, String> getCustomAttributesMap() {
         return attributes;
     }
 
-    public synchronized final void addSecurityRoles(final Collection<String> securityRoles) {
+    public final void addSecurityRoles(final Collection<String> securityRoles) {
         if (securityRoles != null && this.securityRoles != null) {
             this.securityRoles.addAll(securityRoles);
         }
     }
 
-    public synchronized final Set<String> getSecurityRoles() {
+    public final Set<String> getSecurityRoles() {
         return this.securityRoles == null
             ? Collections.synchronizedSet(Collections.emptySet())
             : Collections.unmodifiableSet(this.securityRoles);