fixed formatting and added additional test

Author: ldetmer

google-api-client-apache-v5/src/main/java/com/google/api/client/googleapis/apache/v5/GoogleApache5HttpTransport.java

@@ -47,9 +47,10 @@ public final class GoogleApache5HttpTransport {
 
   /**
    * Returns a new instance of {@link Apache5HttpTransport} that uses default jdk certificates for
-   * the trusted certificates. If `GOOGLE_API_USE_CLIENT_CERTIFICATE` environment variable is set
-   * to "true", and the default client certificate key store from {@link Utils#loadDefaultMtlsKeyStore()}
-   * is not null, then the transport uses the default client certificate and is mutual TLS.
+   * the trusted certificates. If `GOOGLE_API_USE_CLIENT_CERTIFICATE` environment variable is set to
+   * "true", and the default client certificate key store from {@link
+   * Utils#loadDefaultMtlsKeyStore()} is not null, then the transport uses the default client
+   * certificate and is mutual TLS.
    */
   public static Apache5HttpTransport newTrustedTransport()
       throws GeneralSecurityException, IOException {
@@ -58,9 +59,8 @@ public static Apache5HttpTransport newTrustedTransport()
 
   /**
    * {@link Beta} <br>
-   * Returns a new instance of {@link Apache5HttpTransport} that uses default jdk certificates
-   * for the trusted certificates. mtlsProvider can be used to configure mutual TLS for the
-   * transport.
+   * Returns a new instance of {@link Apache5HttpTransport} that uses default jdk certificates for
+   * the trusted certificates. mtlsProvider can be used to configure mutual TLS for the transport.
    *
    * @param mtlsProvider MtlsProvider to configure mutual TLS for the transport
    */

google-api-client-apache-v5/src/test/java/com/google/api/client/googleapis/apache/v5/ITGoogleApache5HttpTransportTest.java

@@ -74,4 +74,23 @@ public Void handleResponse(ClassicHttpResponse response) {
               }
             });
   }
+
+  @Test
+  public void testHttpRequestPassesWhenMakingRequestToSiteContainedInDefaultCerts()
+      throws Exception {
+    Apache5HttpTransport apache5HttpTransport = GoogleApache5HttpTransport.newTrustedTransport();
+    HttpGet httpGet = new HttpGet("https://central.sonatype.com/");
+
+    apache5HttpTransport
+        .getHttpClient()
+        .execute(
+            httpGet,
+            new HttpClientResponseHandler<Void>() {
+              @Override
+              public Void handleResponse(ClassicHttpResponse response) {
+                assertEquals(200, response.getCode());
+                return null;
+              }
+            });
+  }
 }

google-api-client/src/main/java/com/google/api/client/googleapis/apache/v2/GoogleApacheHttpTransport.java

@@ -46,9 +46,10 @@ public final class GoogleApacheHttpTransport {
 
   /**
    * Returns a new instance of {@link ApacheHttpTransport} that uses default jdk certificates for
-   * the trusted certificates. If `GOOGLE_API_USE_CLIENT_CERTIFICATE` environment variable is set
-   * to "true", and the default client certificate key store from {@link Utils#loadDefaultMtlsKeyStore()}
-   * is not null, then the transport uses the default client certificate and is mutual TLS.
+   * the trusted certificates. If `GOOGLE_API_USE_CLIENT_CERTIFICATE` environment variable is set to
+   * "true", and the default client certificate key store from {@link
+   * Utils#loadDefaultMtlsKeyStore()} is not null, then the transport uses the default client
+   * certificate and is mutual TLS.
    */
   public static ApacheHttpTransport newTrustedTransport()
       throws GeneralSecurityException, IOException {
@@ -57,8 +58,8 @@ public static ApacheHttpTransport newTrustedTransport()
 
   /**
    * {@link Beta} <br>
-   * Returns a new instance of {@link ApacheHttpTransport} that default jdk certs for the
-   * trusted certificates. mtlsProvider can be used to configure mutual TLS for the transport.
+   * Returns a new instance of {@link ApacheHttpTransport} that default jdk certs for the trusted
+   * certificates. mtlsProvider can be used to configure mutual TLS for the transport.
    *
    * @param mtlsProvider MtlsProvider to configure mutual TLS for the transport
    */