tweak normalization, allow http with a port through

pinecone/index_connection.go

@@ -1094,12 +1094,18 @@ func sparseValToGrpc(sv *SparseValues) *data.SparseValues {
 }
 
 func normalizeHost(host string) string {
-	// remove http:// or https:// from the host
-	host = strings.TrimPrefix(host, "http://")
+	hasPort := strings.Contains(host, ":")
+
+	// remove https:// from the host
 	host = strings.TrimPrefix(host, "https://")
 
+	// if plaintext without a port, strip http:// as well
+	if !hasPort {
+		host = strings.TrimPrefix(host, "http://")
+	}
+
 	// if a port was provided leave it, otherwise we append :443
-	if !strings.Contains(host, ":") {
+	if !hasPort {
 		host = host + ":443"
 	}
 

pinecone/index_connection_test.go

@@ -1006,9 +1006,13 @@ func TestNormalizeHost(t *testing.T) {
 			host:         "https://this-is-my-host.io",
 			expectedHost: "this-is-my-host.io:443",
 		}, {
-			name:         "port should be maintained",
-			host:         "https://this-is-my-host.io:8080",
-			expectedHost: "this-is-my-host.io:8080",
+			name:         "http:// scheme without a port should be removed",
+			host:         "http://this-is-my-host.io",
+			expectedHost: "this-is-my-host.io:443",
+		}, {
+			name:         "http:// scheme and port should be maintained",
+			host:         "http://this-is-my-host.io:8080",
+			expectedHost: "http://this-is-my-host.io:8080",
 		},
 	}