Port libksieve from ktcpsocket to QSslSocket
ClosedPublic

Authored by mlaurent on Sep 18 2019, 11:12 AM.

Diff Detail

Repository
R91 PIM: Sieve Handling Library
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
mlaurent created this revision.Sep 18 2019, 11:12 AM
Restricted Application added a project: KDE PIM. · View Herald TranscriptSep 18 2019, 11:12 AM
Restricted Application added a subscriber: kde-pim. · View Herald Transcript
mlaurent requested review of this revision.Sep 18 2019, 11:12 AM
vkrause accepted this revision.Sep 18 2019, 11:23 AM

Looks correct to me, very similar to the KSmtp port.

This revision is now accepted and ready to land.Sep 18 2019, 11:23 AM
dfaure added a subscriber: dfaure.Sep 18 2019, 11:56 AM

Confusing commit message. This isn't about port ktcpsocket, but about porting *ksieve* *FROM* ktcpsocket :)

src/kmanagesieve/sessionthread_p.h
55

Hmm, didn't I use this somewhere? Are you sure this compiles?

laurent@linux-zym0:/compile/kde5/framework/kde/pim/libksieve> git grep AuthenticationError
src/kmanagesieve/sessionthread_p.h:55: enum { AuthenticationError = 1500 }; // "extend" KTcpSocket::Error
in master code.

it was never used. I was a dead code :)

mlaurent retitled this revision from Port ktcpsocket to QSslSocket to Port libksieve from ktcpsocket to QSslSocket.Sep 18 2019, 12:12 PM
dfaure accepted this revision.Sep 18 2019, 12:23 PM

Oh oops, indeed. I meant to use it in SessionThread::handleSaslAuthError but in the end it doesn't really matter since we only test for RemoteHostClosedError and SocketTimeoutError. At least currently :)

This revision was automatically updated to reflect the committed changes.