Merge pull request #2599 from sbwalker/dev

add defensive logic to querystring parser to handle duplicate parameters
This commit is contained in:
Shaun Walker 2023-02-15 15:20:12 -05:00 committed by GitHub
commit b3f7353582
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -306,15 +306,21 @@
if (kvp.Contains("=")) if (kvp.Contains("="))
{ {
string[] pair = kvp.Split('='); string[] pair = kvp.Split('=');
if (!querystring.ContainsKey(pair[0]))
{
querystring.Add(pair[0], pair[1]); querystring.Add(pair[0], pair[1]);
} }
}
else else
{
if (!querystring.ContainsKey(kvp))
{ {
querystring.Add(kvp, "true"); // default parameter when no value is provided querystring.Add(kvp, "true"); // default parameter when no value is provided
} }
} }
} }
} }
}
return querystring; return querystring;
} }