3 ms·
The author exploited this by adding a space to the URL so it no longer started with http:// http:// rather (space)http:// http:// but it looks like the call to
by taspeotis 8y ago
The author exploited this by adding a space to the URL so it no longer started with http:// http:// rather (space)http:// http:// but it looks like the call to Replace would be ineffective if the URL started with HTTP:// as well.
bool flag2 = file.Location.ToLower().StartsWith("http://");
if (flag2)
{
file.Location = file.Location.Replace("http://", "https://");
}
I trust the new version isn’t vulnerable to this...
- BillDemirkapi 8y agoThere were a bunch of ways to bypass the check. For example another way would be to use "http:\\" which wouldn't get detected either. The new version isn't vulnerable.