Merge pull request #2744 from midnight-wonderer/bug/hostname-matching

PhilippC · web-flow · commit 07562cc5a967 · 2025-02-11T12:33:49.000+01:00
Fix hostname matching logic. closes #1926
diff --git a/src/Kp2aBusinessLogic/SearchDbHelper.cs b/src/Kp2aBusinessLogic/SearchDbHelper.cs
@@ -138,7 +138,8 @@ public PwGroup SearchForHost(Database database, String url, bool allowSubdomains
 				{
 					continue;
 				}
-				if (host.IndexOf(otherHost, StringComparison.InvariantCultureIgnoreCase) > -1)
+				if (string.Equals(host, otherHost, StringComparison.OrdinalIgnoreCase) ||
+					host.EndsWith("." + otherHost, StringComparison.OrdinalIgnoreCase))
 				{
 					pgResults.AddEntry(entry, false);
 				}

Original file line number	Diff line number	Diff line change
`@@ -138,7 +138,8 @@ public PwGroup SearchForHost(Database database, String url, bool allowSubdomains`
`138`	`138`	`{`
`139`	`139`	`continue;`
`140`	`140`	`}`
`141`		`- if (host.IndexOf(otherHost, StringComparison.InvariantCultureIgnoreCase) > -1)`
	`141`	`+ if (string.Equals(host, otherHost, StringComparison.OrdinalIgnoreCase) \|\|`
	`142`	`+ host.EndsWith("." + otherHost, StringComparison.OrdinalIgnoreCase))`
`142`	`143`	`{`
`143`	`144`	`pgResults.AddEntry(entry, false);`
`144`	`145`	`}`