Auto-generated PAC file: improve based on SwitchyOmega practices about forwardproxy HOT 4 CLOSED

This seems straightforward. The relevant line is

if (/^127\.0\.0\.1$/.test(host) || /^::1$/.test(host) || /^localhost$/.test(host)) return "DIRECT";

We could add a similar check to our PAC file and it should work fine. (Although in our code I think we would use string equality, not this odd use of regex matching, which is probably the output of a code generator.)

from forwardproxy.

Yes, this part does seem straightforward, and good point that string equality would be better.
By "sorcery", I meant first half of this function, including function signature: I am not very proficient in js.

from forwardproxy.

That boilerplate is just for allowing their codegen system to make choices based on scheme, and to support multiple profiles. It preprocesses the url to find the scheme, and looks up the policy function by profile name. None of that is relevant to the goal of excluding localhost, which we can do just by adding that one line to return "DIRECT".

from forwardproxy.

Thanks for help!

from forwardproxy.