Fix: Removed null check from getDefaultRequestConfigWithTimeout to create separate request configs (#498)

Author: mnoman09

core-httpclient-impl/src/main/java/com/optimizely/ab/HttpClientUtils.java

@@ -1,6 +1,6 @@
 /**
  *
- *    Copyright 2016-2017, 2019, Optimizely and contributors
+ *    Copyright 2016-2017, 2019, 2022-2023, Optimizely and contributors
  *
  *    Licensed under the Apache License, Version 2.0 (the "License");
  *    you may not use this file except in compliance with the License.
@@ -39,13 +39,11 @@ private HttpClientUtils() {
         .build();
 
     public static RequestConfig getDefaultRequestConfigWithTimeout(int timeoutMillis) {
-        if (requestConfigWithTimeout == null) {
-            requestConfigWithTimeout = RequestConfig.custom()
-                .setConnectTimeout(timeoutMillis)
-                .setConnectionRequestTimeout(CONNECTION_REQUEST_TIMEOUT_MS)
-                .setSocketTimeout(SOCKET_TIMEOUT_MS)
-                .build();
-        }
+        requestConfigWithTimeout = RequestConfig.custom()
+            .setConnectTimeout(timeoutMillis)
+            .setConnectionRequestTimeout(CONNECTION_REQUEST_TIMEOUT_MS)
+            .setSocketTimeout(timeoutMillis)
+            .build();
         return  requestConfigWithTimeout;
     }