<table><tr><td style="">marten added inline comments.
</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/D21463">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D21463#inline-120854">View Inline</a><span style="color: #4b4d51; font-weight: bold;">dvratil</span> wrote in <span style="color: #4b4d51; font-weight: bold;">sessionthread.cpp:68</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Unless the socket is disconnected, you should probably force reconnection here to apply the new configuration...</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Is this needed?  I've traced the agent and the execution of the SessionThread, and if the account settings are changed in KMail the agent is set offline then online.  This means that threadQuit() and threadInit() are called which recreates the socket and reconnects to the server with the new settings.</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D21463#inline-120853">View Inline</a><span style="color: #4b4d51; font-weight: bold;">dvratil</span> wrote in <span style="color: #4b4d51; font-weight: bold;">sessionthread.cpp:180</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">More context: "Connecting to IMAP server using default system proxy"</p>

<p style="padding: 0; margin: 8px;">Shouldn't you also explicitly set the <tt style="background: #ebebeb; font-size: 13px;">QNetworkProxy</tt> here as well? The socket doesn't get recreated on reconnect, so if the option is changed, it wouldn't get reset.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Set here to make sure, but as above the socket and connection is recreated when the option is changed by the user anyway.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R177 PIM: KIMAP</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D21463">https://phabricator.kde.org/D21463</a></div></div><br /><div><strong>To: </strong>marten, KDE PIM, mlaurent, dvratil<br /><strong>Cc: </strong>dvratil, mlaurent, kde-pim, dvasin, rodsevich, winterz, vkrause, knauss<br /></div>