Bug 69611 – [Qt] WebProcess using proxy settings for localhost as well

• Component: WebKit2
• Status: RESOLVED FIXED

[Laszlo Gombos]

http://localhost should not go trough proxy even if http_proxy environment variable is set. It seems that the current solution does not take localhost/127.0.0.1 into account; see http://trac.webkit.org/changeset/72080/trunk/WebKit2/WebProcess/qt/WebProcessMainQt.cpp.

[Gopal Raghavan]

Created attachment 110366 [details]
proxy fix for localhost

Implemented checks in queryProxy to return QNetworkProxy::NoProxy for localhost and 127.0.0.1.
Not sure where the test cases are for proxy settings. If you have any suggestion, I can add a test case.

[Chang Shu]

Comment on attachment 110366 [details]
proxy fix for localhost

Excellent patch! For perfection, you can possibly explore the option of replacing the "localhost" and "127.0.0.1" with some redefined Qt variables and whether we should go "CaseSensitive". But the patch is good to go.

[Gopal Raghavan]

I grepped through qt5 refactored. I couldn't find a predefined for localhost. There are many occurrences of string. There is an enum under QHostAddress, but that might not be useful in this case.

Chrome, firefox etc., accept localhost and LOCALHOST. So, I think CaseInSensitive should be ok. Ofcourse the numbers dont need it.
Thanks

[WebKit Review Bot]

Comment on attachment 110366 [details]
proxy fix for localhost

Clearing flags on attachment: 110366

Committed r97154: <http://trac.webkit.org/changeset/97154>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.