Fix proxy KCM not loading manual proxies correctly.
ClosedPublic

Authored by dfaure on Aug 4 2017, 7:50 PM.

Details

Summary

commit 4e2a52c671 prepends a scheme (like socks://), which interfers with
the assumption that after the first ':' comes the port.
But we can use QUrl parsing to extract the port number, instead.

Test Plan

kcmshell5 proxy, setting manual proxy values, OK, kcmshell5 proxy

Diff Detail

Repository
R241 KIO
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
dfaure created this revision.Aug 4 2017, 7:50 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptAug 4 2017, 7:50 PM
davidedmundson accepted this revision.Aug 4 2017, 9:50 PM
This revision is now accepted and ready to land.Aug 4 2017, 9:50 PM
dfaure closed this revision.Aug 7 2017, 10:31 PM