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

[Dawit A]

On Wed, Mar 9, 2011 at 12:46 PM, David Faure <faure at kde.org> wrote:
> On Wednesday 09 March 2011, Dawit Alemayehu wrote:
>> proxiedBy always returns true since ftp.protocol no longer contains a
>> "ProxiedBy=" entry.
>
> Yes it does, of course. But the point of the additional if() is to not go into
> that code block for other procotols, like HTTP, FISH, and so on.

I meant to say "always returns false". KProtocolInfo::proxiedBy(...)
always returns an empty string for FTP. I just looked at the log for
ftp.protocol file and it seems that Andreas had mistakenly removed the
"proxiedBy=" statement from the ftp.protocol file. See

http://quickgit.kde.org/?p=kdelibs.git&a=commit&h=7b61621c023db80af888a12be7fcb9fd6c9a6fb8

As such the changes the fix I committed current does not fix the
problem. FTP over HTTP proxy still does not work. Anyhow, if the
incorrect commit mentioned above gets reverted, the problem away. Any
objections with me doing that ?

Regards,
Dawit A.