Add http prefix cleaning

Author: Clint Checketts

domo-java-sdk-all/src/main/java/com/domo/sdk/request/Config.java

@@ -2,13 +2,16 @@
 
 import okhttp3.OkHttpClient;
 import okhttp3.logging.HttpLoggingInterceptor;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
 import java.util.concurrent.TimeUnit;
 
 public class Config {
+    private static final Logger LOG = LoggerFactory.getLogger(Config.class);
     private final String clientId;
     private final String secret;
     private final List<Scope> scopes;
@@ -27,7 +30,7 @@ public Config(String clientId,
         this.clientId = clientId;
         this.secret = secret;
         this.scopes = Collections.unmodifiableList(scopes);
-        this.apiHost = apiHost;
+        this.apiHost = stripPrefix(apiHost);
         this.useHttps = useHttps;
 
         if(httpLoggingLevel == null) {
@@ -50,6 +53,20 @@ public Config(String clientId,
 
     }
 
+    // Visible for testing.
+    String stripPrefix(String apiHost) {
+        String httpPrefix = "http://";
+        String httpsPrefix = "https://";
+        if(apiHost.toLowerCase().startsWith(httpPrefix)) {
+            LOG.warn("Ignoring http hpiHost scheme, set 'useHttps' to false for http");
+            return apiHost.substring(httpPrefix.length());
+        } else if(apiHost.toLowerCase().startsWith(httpsPrefix)) {
+            return apiHost.substring(httpsPrefix.length());
+        } else {
+            return apiHost;
+        }
+    }
+
     public String getClientId() {
         return clientId;
     }

domo-java-sdk-all/src/test/java/com/domo/sdk/request/ConfigTest.java

@@ -0,0 +1,26 @@
+package com.domo.sdk.request;
+
+import okhttp3.logging.HttpLoggingInterceptor;
+import org.junit.Test;
+
+import java.util.ArrayList;
+
+import static org.assertj.core.api.Assertions.*;
+
+public class ConfigTest {
+
+  @Test
+  public void apiHost_shouldRemoveScheme() {
+    Config c = newConfig("hostwithhttp.com:1234");
+    assertThat(c.getApiHost()).isEqualTo("hostwithhttp.com:1234");
+
+    assertThat( newConfig("http://example.com").getApiHost()).isEqualTo("example.com");
+    assertThat( newConfig("https://other.com:1234").getApiHost()).isEqualTo("other.com:1234");
+
+  }
+
+  private Config newConfig(String host) {
+    return new Config("a", "b", host, true, new ArrayList<>(), null, HttpLoggingInterceptor.Level.BASIC);
+  }
+
+}