Ensure upload base path ends in /v1

Signed-off-by: Dan Luhring <dan.luhring@anchore.com>

internal/anchore/client.go

@@ -31,6 +31,9 @@ func NewClient(cfg Configuration) (*Client, error) {
 
 	basePath := ensureURLHasScheme(cfg.BasePath) // we can rely on the built-in URL parsing for the scheme, host,
 	// port, and path prefix, as long as a scheme is present
+	basePath = strings.TrimSuffix(basePath, "/")
+	basePath = ensureURLHasSuffix(basePath,
+		"/v1") // We need some mechanism to ensure Syft doesn't try to communicate with the wrong API version.
 
 	return &Client{
 		config: cfg,
@@ -70,3 +73,11 @@ func ensureURLHasScheme(url string) string {
 
 	return url
 }
+
+func ensureURLHasSuffix(url, suffix string) string {
+	if !strings.HasSuffix(url, suffix) {
+		return url + suffix
+	}
+
+	return url
+}

internal/anchore/client_test.go

@@ -69,3 +69,41 @@ func TestEnsureURLHasScheme(t *testing.T) {
 		})
 	}
 }
+func TestEnsureURLHasSuffix(t *testing.T) {
+	cases := []struct {
+		url      string
+		suffix   string
+		expected string
+	}{
+		{
+			url:      "http://localhost",
+			suffix:   "/v1",
+			expected: "http://localhost/v1",
+		},
+		{
+			url:      "http://localhost/v1",
+			suffix:   "/v1",
+			expected: "http://localhost/v1",
+		},
+		{
+			url:      "http://localhost/v1/",
+			suffix:   "/v1",
+			expected: "http://localhost/v1//v1",
+		},
+		{
+			url:      "http://localhost-v1",
+			suffix:   "/v1",
+			expected: "http://localhost-v1/v1",
+		},
+	}
+
+	for _, testCase := range cases {
+		t.Run(testCase.url, func(t *testing.T) {
+			result := ensureURLHasSuffix(testCase.url, testCase.suffix)
+
+			if testCase.expected != result {
+				t.Errorf("expected '%s' but got '%s'", testCase.expected, result)
+			}
+		})
+	}
+}