Review Request 113364: kio_http: fix for proxy connection

[Andrea Iacovitti]

> On Oct. 25, 2013, 2:55 a.m., Dawit Alemayehu wrote:
> > I have no objection to this patch. However, I want to make sure you have tested the other modes before I give it a thumbs up. Did you try the manual proxy setup with your patch? Also adding some exceptions to make sure it honors those as well? If not can you please do that I do not have the bandwidth to test that at the moment.

I did already and it works. But (correct me if i'm wrong) if it would not then the issue should be in KProtocolManager that makes available to kio_http the list of proxies to evaluate. 
This patch only fixes how that list is parsed.


- Andrea


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113364/#review42328
-----------------------------------------------------------


On Oct. 21, 2013, 1:41 p.m., Andrea Iacovitti wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113364/
> -----------------------------------------------------------
> 
> (Updated Oct. 21, 2013, 1:41 p.m.)
> 
> 
> Review request for kdelibs and Dawit Alemayehu.
> 
> 
> Repository: kdelibs
> 
> 
> Description
> -------
> 
> Fix the code to determine if a proxy or direct connection should be made in the case where multiple proxies are availables and between them the keyword "DIRECT" is present. We should first verify if the candidate proxy string is equal to "DIRECT" (and then try to make a direct connection) and after try to construct KUrl object.
> Otherwise we would never be able to direct connect because KUrl("DIRECT") is not a valid (proxy) url.
> 
> 
> Diffs
> -----
> 
>   kioslave/http/http.cpp 81182eb 
> 
> Diff: http://git.reviewboard.kde.org/r/113364/diff/
> 
> 
> Testing
> -------
> 
> Tested against various PAC scripts returning, for example, proxies list like:
> ("http://127.0.0.1:3333", "DIRECT") or ("DIRECT", "http://127.0.0.1:3333")
> 
> Note: in case PAC script returns "DIRECT" only, this code path is not executed as m_request.proxyUrls will be empty.
> 
> 
> Thanks,
> 
> Andrea Iacovitti
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20131025/ae59489a/attachment.htm>