Store installationTokens in a ConcurrentHashMap (spotify#195)

mitchhentgesspotify · web-flow · commit 8a9f799cd67a · 2024-07-10T15:38:10.000+02:00
Hopefully this is sufficient to make `GitHubClient` thread-safe.

No other usages of `HashMap` or `ArrayList` were found, except for
`Languages` which is a return structure, so is less
necessary to be thread-safe.
diff --git a/src/main/java/com/spotify/github/v3/clients/GitHubClient.java b/src/main/java/com/spotify/github/v3/clients/GitHubClient.java
@@ -48,12 +48,12 @@
 import java.lang.invoke.MethodHandles;
 import java.net.URI;
 import java.time.ZonedDateTime;
-import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 import java.util.Objects;
 import java.util.Optional;
 import java.util.concurrent.CompletableFuture;
+import java.util.concurrent.ConcurrentHashMap;
 import java.util.concurrent.atomic.AtomicBoolean;
 import java.util.function.Consumer;
 import javax.ws.rs.core.HttpHeaders;
@@ -145,7 +145,7 @@ private GitHubClient(
     this.privateKey = privateKey;
     this.appId = appId;
     this.installationId = installationId;
-    this.installationTokens = new HashMap<>();
+    this.installationTokens = new ConcurrentHashMap<>();
   }
 
   /**
