Fix for issue #106 as discussed

common/src/main/java/com/google/tsunami/common/net/http/HttpClient.java

@@ -93,8 +93,6 @@ public abstract static class Builder<T extends HttpClient> {
 
     public abstract Builder<T> setFollowRedirects(boolean followRedirects);
 
-    public abstract Builder<T> setTrustAllCertificates(boolean trustAllCertificates);
-
     public abstract Builder<T> setLogId(String logId);
 
     public abstract Builder<T> setConnectTimeout(Duration connectionTimeout);

common/src/main/java/com/google/tsunami/common/net/http/HttpClientCliOptions.java

@@ -27,6 +27,7 @@ public final class HttpClientCliOptions implements CliOption {
 
   @Parameter(
       names = "--http-client-trust-all-certificates",
+      arity = 1,
       description = "Whether the HTTP client should trust all certificates on HTTPS traffic.")
   public Boolean trustAllCertificates;
 

common/src/main/java/com/google/tsunami/common/net/http/HttpClientModule.java

@@ -171,7 +171,7 @@ boolean shouldTrustAllCertificates(
     if (httpClientConfigProperties.trustAllCertificates != null) {
       return httpClientConfigProperties.trustAllCertificates;
     }
-    return false;
+    return true;
   }
 
   @Provides

common/src/main/java/com/google/tsunami/common/net/http/OkHttpHttpClient.java

@@ -374,12 +374,6 @@ public OkHttpHttpClientBuilder setFollowRedirects(boolean followRedirects) {
       return this;
     }
 
-    @Override
-    public OkHttpHttpClientBuilder setTrustAllCertificates(boolean trustAllCertificates) {
-      this.trustAllCertificates = trustAllCertificates;
-      return this;
-    }
-
     @Override
     public OkHttpHttpClientBuilder setLogId(String logId) {
       this.logId = logId;