Merge pull request #168 from notsure2/fix-empty-alternative-names

Fix incorrect addition of empty name in MockDomainList if the config string contains AlternativeNames=;
cbeuw · Jul 24, 2021 · 121c27f · 121c27f
2 parents d73f464 + 9d5c663
commit 121c27f
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/internal/client/state.go b/internal/client/state.go
@@ -154,6 +154,15 @@ func (raw *RawConfig) ProcessRawConfig(worldState common.WorldState) (local Loca
 		return nullErr("ServerName")
 	}
 	auth.MockDomain = raw.ServerName
+
+	var filteredAlternativeNames []string
+	for _, alternativeName := range raw.AlternativeNames {
+		if len(alternativeName) > 0 {
+			filteredAlternativeNames = append(filteredAlternativeNames, alternativeName)
+		}
+	}
+	raw.AlternativeNames = filteredAlternativeNames
+
 	local.MockDomainList = raw.AlternativeNames
 	local.MockDomainList = append(local.MockDomainList, auth.MockDomain)
 	if raw.ProxyMethod == "" {