Additional adjustments to the query string forwarding logic

In earlier versions of this package the query string forwarding logic were primarily handled by querying the database, which was case insensitive. After moving some of the logic to C#, this part was somewhat unintentionally case sensitive, while the part still handled in the database was still case insensitive. With this commit both the database query and the C# logic are now case sensitive to better match the previous behavior.

Whether it should in fact be case insensitive is however a whole other question. It probably shouldn't be, but it has been like this ever since the first release as SQL WHERE clauses are case insensitive. So we shouldn't change this behavior now.

Related to #193

(cherry picked from commit 542a4a8)

Author: abjerner

src/Skybrud.Umbraco.Redirects/Services/RedirectsService.cs

@@ -152,7 +152,7 @@ public void DeleteRedirect(IRedirect redirect) {
             // - query string forwarding isn't enabled and the query string is an exact match
             // - query string forwarding is enabled and the query string is part of the query string of the inbound URI
             string query1 = query.Length == 0 ? string.Empty : $"&{query}&";
-            RedirectDto? dto = dtos.FirstOrDefault(x => (!x.ForwardQueryString && query == x.QueryString) || (x.QueryString.Length == 0 || query1.Contains($"&{x.QueryString}&") && x.ForwardQueryString));
+            RedirectDto? dto = dtos.FirstOrDefault(x => (!x.ForwardQueryString && query.InvariantEquals(x.QueryString)) || (x.QueryString.Length == 0 || query1.InvariantContains($"&{x.QueryString}&") && x.ForwardQueryString));
 
             // Wrap the DTO
             return dto == null ? null : new Redirect(dto);