undefined | Better HN

0 pointsGordonS6y ago0 comments

A naive regex would just as easily have exactly the same issue, e.g. "^http:"

0 comments

That wouldn't have the same issue since the space at the beginning would fail that regex.

I believe that's parent's point. "https:" would be OK, "http: would be rejected, but " http:" would also be _accepted_ it. They looked for "http://" at the start of the string, instead of requiring "https://".

Replacing:

    bool flag2 = file.Location.ToLower().StartsWith("http://");

with:

    bool flag2 = Regex.IsMatch(file.Location.ToLower(), "^http:");

doesn't help. You have to make sure to actually replace http, not just check the start of the line.

j / k navigate · click thread line to collapse