Review Request 125904: Save proxy url with correct scheme.
Xuetian Weng
wengxt at gmail.com
Tue Nov 3 15:48:41 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125904/
-----------------------------------------------------------
(Updated Nov. 3, 2015, 3:48 p.m.)
Review request for KDE Frameworks and Eike Hein.
Repository: kio
Description
-------
Currently, if user type a ip address/domain name in socks 5 proxy field, the saved proxy url in configuration file will start with http:// instead of socks:// . Same applies to ftp proxy.
The reason is that KUrlFilter append http scheme to url without scheme. This patch uses setDefaultUrlScheme to make sure the correct scheme is used.
Also make necessary change to make sure that loading url configuration will hide url scheme correctly.
Diffs (updated)
-----
src/kcms/kio/kproxydlg.cpp c8fd0cd
Diff: https://git.reviewboard.kde.org/r/125904/diff/
Testing
-------
Save socks proxy and ftp proxy will result in "ftp://" and "socks://".
Thanks,
Xuetian Weng
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20151103/c920d4e4/attachment.html>
More information about the Kde-frameworks-devel
mailing list