Revert attributes map back to synchronizedMap.

The-inside-man · The-inside-man · commit cef2c13a7182 · 2021-11-03T13:14:19.000-04:00
diff --git a/core-api/src/main/java/com/optimizely/ab/OptimizelyUserContext.java b/core-api/src/main/java/com/optimizely/ab/OptimizelyUserContext.java
@@ -60,9 +60,9 @@ public OptimizelyUserContext(@Nonnull Optimizely optimizely,
         this.optimizely = optimizely;
         this.userId = userId;
         if (attributes != null) {
-            this.attributes = new ConcurrentHashMap<>(new HashMap<>(attributes));
+            this.attributes = Collections.synchronizedMap(new HashMap<>(attributes));
         } else {
-            this.attributes = new ConcurrentHashMap<>(new HashMap<>());
+            this.attributes = Collections.synchronizedMap(new HashMap<>());
         }
         this.forcedDecisionsMap = forcedDecisionsMap != null ? forcedDecisionsMap : new ConcurrentHashMap<>();
     }

Original file line number	Diff line number	Diff line change
`@@ -60,9 +60,9 @@ public OptimizelyUserContext(@Nonnull Optimizely optimizely,`
`60`	`60`	`this.optimizely = optimizely;`
`61`	`61`	`this.userId = userId;`
`62`	`62`	`if (attributes != null) {`
`63`		`- this.attributes = new ConcurrentHashMap<>(new HashMap<>(attributes));`
	`63`	`+ this.attributes = Collections.synchronizedMap(new HashMap<>(attributes));`
`64`	`64`	`} else {`
`65`		`- this.attributes = new ConcurrentHashMap<>(new HashMap<>());`
	`65`	`+ this.attributes = Collections.synchronizedMap(new HashMap<>());`
`66`	`66`	`}`
`67`	`67`	`this.forcedDecisionsMap = forcedDecisionsMap != null ? forcedDecisionsMap : new ConcurrentHashMap<>();`
`68`	`68`	`}`