Classify GITHUB_HOST case insensitively

Signed-off-by: Sebastian Turner <sebastiant3484@gmail.com>

Author: sebxt

pkg/utils/api.go

@@ -301,10 +301,12 @@ const (
 )
 
 func classifyHost(u *url.URL) HostType {
+	// Hostnames are case-insensitive, so "https://GitHub.com" is still github.com.
+	hostname := strings.ToLower(u.Hostname())
 	switch {
-	case u.Hostname() == "github.com" || strings.HasSuffix(u.Hostname(), ".github.com"):
+	case hostname == "github.com" || strings.HasSuffix(hostname, ".github.com"):
 		return HostTypeDotcom
-	case u.Hostname() == "ghe.com" || strings.HasSuffix(u.Hostname(), ".ghe.com"):
+	case hostname == "ghe.com" || strings.HasSuffix(hostname, ".ghe.com"):
 		return HostTypeGHEC
 	default:
 		return HostTypeGHES

pkg/utils/api_test.go

@@ -55,6 +55,16 @@ func TestParseAPIHost(t *testing.T) {
 			input:       "https://myghe.com",
 			wantRestURL: "https://myghe.com/api/v3/",
 		},
+		{
+			name:        "mixed case github.com hostname",
+			input:       "https://GitHub.com",
+			wantRestURL: "https://api.github.com/",
+		},
+		{
+			name:        "mixed case ghe.com subdomain",
+			input:       "https://Contoso.GHE.com",
+			wantRestURL: "https://api.Contoso.GHE.com/",
+		},
 		{
 			name:    "missing scheme",
 			input:   "github.com",