<table><tr><td style="">sitter updated this revision to Diff 67695.<br />sitter marked 24 inline comments as done.<br />sitter added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D23579">View Revision</a></tr></table><br /><div><div><ul class="remarkup-list">
<li class="remarkup-list-item">rebase</li>
<li class="remarkup-list-item">ftpSendCmd now returns a bool (continues to require_result)</li>
<li class="remarkup-list-item">openConnection now emits opened()</li>
<li class="remarkup-list-item">listDir now finalize()s</li>
<li class="remarkup-list-item">debug--</li>
<li class="remarkup-list-item">ftpSendCmd's lazy login now checks the return value of the recrusive cmd call</li>
</ul>

<p>ftpOpenControlConnection's proxy handling is completely redone:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">loosened nesting</li>
<li class="remarkup-list-item">socks5 is now also supported as scheme</li>
<li class="remarkup-list-item">reliance on eventloop is gone and replaced with actually sync behavior for the sync synchronousConnectToHost<ul class="remarkup-list">
<li class="remarkup-list-item">the way proxy management is done is no longer using applicationproxy but instead managing the proxy on the socket(s) directly. this makes things somewhat more explicit and gives less opportunity for errors inside Qt behavior</li>
<li class="remarkup-list-item">now returns a composite Result type to carry both context and socket to the caller. contextual error forwarding is handy here because there is value in differntiating why authentication isn't working when e.g. kiod is defunct.</li>
<li class="remarkup-list-item">qauthenticator use gone as it has no api outside the auth signal</li>
<li class="remarkup-list-item">proxyAuthentication and saveProxyAuthentication have been merged into synchronousConnectToHost and been simplified<ul class="remarkup-list">
<li class="remarkup-list-item">now only uses one AuthInfo object</li>
<li class="remarkup-list-item">realm is no longer set (came from QAuthenticator but as far as I can tell the QA created by QAbstractSocket has no realm set, so this never did anything anyway)</li>
<li class="remarkup-list-item">as a result the comment field in password queries is now only showing the hostname instead of a broken " at $hostname"</li>
</ul></li>
</ul></li>
<li class="remarkup-list-item">added ProxyTesting.md with some guidance on how to test this stuff</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D23579?vs=66747&id=67695">https://phabricator.kde.org/D23579?vs=66747&id=67695</a></div></div><br /><div><strong>BRANCH</strong><div><div>ftp</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D23579">https://phabricator.kde.org/D23579</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>autotests/CMakeLists.txt<br />
autotests/ftp/testCopy1<br />
autotests/ftp/testCopy2<br />
autotests/ftp/testOverwriteCopy1<br />
autotests/ftp/testOverwriteCopy2<br />
autotests/ftpd<br />
autotests/ftptest.cpp<br />
cmake/FindGem.cmake<br />
cmake/FindGem.cmake.in<br />
cmake/FindRubyExe.cmake<br />
src/ioslaves/ftp/ftp.cpp<br />
src/ioslaves/ftp/ftp.h</div></div></div><br /><div><strong>To: </strong>sitter, dfaure<br /><strong>Cc: </strong>anthonyfieroni, dfaure, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>