Remove code specific to Internet Explorer

[theseion]

The code includes some special cases for Internet Explorer. Microsoft doesn't officially support it anymore. Users still exist, usually in legacy corporate intranet environments, but they can be considered a special case.

I propose to either

• remove code that is specific to Internet Explorer, or
• put it behind a compiler flag

Removing this legacy code is not simply about removing technical debt, at least one code snippet actually impacts every request, where a check need to be handled differently just because of a quirk of Internet Explorer. The removal of that code would thus benefit all users.

[airween]

Hi @theseion,

thanks for bringing this up - could you give some more explanation about the details? I mean versions (v2/v3), and/or a few examples, how can we find these hacks (eg do they have some spec comments?)

Thanks!

[theseion]

I saw this comment during PR review. That's the only thing I saw in v3.

The same piece of code is present in v2.

That's all I found. There are other matches in tests and libinjection, but I don't think we have to do anything about those.