fix: ApiClientUtils.getDefaultTransport() should return cached instance to an ApacheHttp2Transport

Author: jonathanedey

src/main/java/com/google/firebase/internal/ApiClientUtils.java

@@ -88,6 +88,10 @@ public static JsonFactory getDefaultJsonFactory() {
   }
 
   public static HttpTransport getDefaultTransport() {
-    return new ApacheHttp2Transport();
+    return TransportInstanceHolder.INSTANCE;
+  }
+
+  private static class TransportInstanceHolder {
+    static final HttpTransport INSTANCE = new ApacheHttp2Transport();
   }
 }

src/test/java/com/google/firebase/internal/ApacheHttp2TransportIT.java

@@ -292,6 +292,21 @@ public void testVerifyProxyIsRespected() {
     }
   }
 
+  @Test
+  public void testVerifyDefaultTransportReused() {
+    FirebaseOptions o1 = FirebaseOptions.builder()
+      .setCredentials(MOCK_CREDENTIALS)
+      .build();
+    
+    FirebaseOptions o2 = FirebaseOptions.builder()
+      .setCredentials(MOCK_CREDENTIALS)
+      .build();
+
+    HttpTransport t1 = o1.getHttpTransport();
+    HttpTransport t2 = o2.getHttpTransport();
+    assertEquals(t1, t2);
+  }
+
   private static ErrorHandlingHttpClient<FirebaseException> getHttpClient(boolean authorized,
       FirebaseApp app) {
     HttpRequestFactory requestFactory;

src/test/java/com/google/firebase/internal/ApiClientUtilsTest.java

@@ -25,6 +25,7 @@
 import com.google.api.client.http.HttpRequest;
 import com.google.api.client.http.HttpRequestFactory;
 import com.google.api.client.http.HttpResponse;
+import com.google.api.client.http.HttpTransport;
 import com.google.api.client.http.HttpUnsuccessfulResponseHandler;
 import com.google.api.client.testing.http.MockHttpTransport;
 import com.google.api.client.testing.http.MockLowLevelHttpResponse;
@@ -128,4 +129,12 @@ public void disconnect() throws IOException {
 
     assertTrue(lowLevelResponse.isDisconnected());
   }
+
+  @Test
+  public void testVerifyDefaultTransportReused() {
+    HttpTransport t1 = ApiClientUtils.getDefaultTransport();
+    HttpTransport t2 = ApiClientUtils.getDefaultTransport();
+    assertEquals(t1, t2);
+  }
+
 }