Review Request: Fix for connecting to FTP sites through HTTP proxy

David Faure faure at kde.org
Tue Mar 8 09:28:48 GMT 2011


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

Ship it!


Thanks.

- David


On March 8, 2011, 12:43 a.m., Dawit Alemayehu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100814/
> -----------------------------------------------------------
> 
> (Updated March 8, 2011, 12:43 a.m.)
> 
> 
> Review request for kdelibs.
> 
> 
> Summary
> -------
> 
> The attached patch fixes accessing FTP sites through HTTP proxy servers for KDE based browsers, such as Konqueror, only. Why only browsers ? Because FTP over HTTP proxy sends back HTML pages and not file items. As such, the returned result cannot be shown in Dolphin or any application or dialog that relies on the dolphin part.
> 
> There is however a much larger issue here that this patch does not address. How to support connections to FTP sites in all apps/dialogs that use the dolphinpart when such requests are routed through an HTTP proxy ? There is a possibility of using proxy tunneling like it is done for HTTPS over HTTP proxy. However, that would require kio_ftp being changed to support connecting to an HTTP proxy server. Not a minor change. The other option I can think of is to modify the dolphinpart to parse the HTML file returned in such cases and create file items out of them, but that too is not a complicated solution.
> 
> Anyhow, regardless of what could or could not be done to resolve that issue in the future, I think that the dolphinpart should at least should check to see if the ftp request is going over proxy (KProtocolManager::slaveProtocol) and inform the user such action is not supported.
> 
> 
> This addresses bug 236630.
>     http://bugs.kde.org/show_bug.cgi?id=236630
> 
> 
> Diffs
> -----
> 
>   kioslave/http/http.cpp 0579382 
>   kparts/browserrun.cpp 3a062d2 
> 
> Diff: http://git.reviewboard.kde.org/r/100814/diff
> 
> 
> Testing
> -------
> 
> Used Konqueror to connect to FTP sites through a local proxy server.
> 
> 
> Thanks,
> 
> Dawit
> 
>

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


More information about the kde-core-devel mailing list