Detect login failures

3 files changed, 16 insertions, 7 deletions

diff --git a/examples/imapresource/imapresource.cpp b/examples/imapresource/imapresource.cpp
index fde8260..14c3a3e 100644
--- a/examples/imapresource/imapresource.cpp
+++ b/examples/imapresource/imapresource.cpp
@@ -509,6 +509,8 @@ public:
                     return {ApplicationDomain::ConnectionError, error.errorMessage};
                 case Imap::SslHandshakeError:
                     return {ApplicationDomain::LoginError, error.errorMessage};
+                case Imap::LoginFailed:
+                    return {ApplicationDomain::LoginError, error.errorMessage};
                 case Imap::HostNotFoundError:
                     return {ApplicationDomain::NoServerError, error.errorMessage};
                 case Imap::ConnectionLost:
diff --git a/examples/imapresource/imapserverproxy.cpp b/examples/imapresource/imapserverproxy.cpp
index 317fbdc..9a84327 100644
--- a/examples/imapresource/imapserverproxy.cpp
+++ b/examples/imapresource/imapserverproxy.cpp
@@ -62,11 +62,9 @@ const char* Imap::Capabilities::Namespace = "NAMESPACE";
 const char* Imap::Capabilities::Uidplus = "UIDPLUS";
 const char* Imap::Capabilities::Condstore = "CONDSTORE";
 
-static int translateImapError(int error)
+static int translateImapError(int error, bool isLoginJob)
 {
     switch (error) {
-        case KJob::UserDefinedError:
-            return Imap::ConnectionLost;
         case KIMAP2::LoginJob::ErrorCode::ERR_HOST_NOT_FOUND:
             return Imap::HostNotFoundError;
         case KIMAP2::LoginJob::ErrorCode::ERR_COULD_NOT_CONNECT:
@@ -74,6 +72,14 @@ static int translateImapError(int error)
         case KIMAP2::LoginJob::ErrorCode::ERR_SSL_HANDSHAKE_FAILED:
             return Imap::SslHandshakeError;
     }
+    //Hack to detect login failures
+    if (isLoginJob) {
+        return Imap::LoginFailed;
+    }
+    //Hack to detect connection lost
+    if (error == KJob::UserDefinedError) {
+        return Imap::ConnectionLost;
+    }
     return Imap::UnknownError;
 }
 
@@ -84,8 +90,8 @@ static KAsync::Job<T> runJob(KJob *job, const std::function<T(KJob*)> &f)
         QObject::connect(job, &KJob::result, [&future, f](KJob *job) {
             SinkTrace() << "Job done: " << job->metaObject()->className();
             if (job->error()) {
-                SinkWarning() << "Job failed: " << job->errorString() << job->metaObject()->className();
-                auto proxyError = translateImapError(job->error());
+                SinkWarning() << "Job failed: " << job->errorString() << job->metaObject()->className() << job->error();
+                auto proxyError = translateImapError(job->error(), dynamic_cast<KIMAP2::LoginJob*>(job));
                 future.setError(proxyError, job->errorString());
             } else {
                 future.setValue(f(job));
@@ -103,8 +109,8 @@ static KAsync::Job<void> runJob(KJob *job)
         QObject::connect(job, &KJob::result, [&future](KJob *job) {
             SinkTrace() << "Job done: " << job->metaObject()->className();
             if (job->error()) {
-                SinkWarning() << "Job failed: " << job->errorString() << job->metaObject()->className();
-                auto proxyError = translateImapError(job->error());
+                SinkWarning() << "Job failed: " << job->errorString() << job->metaObject()->className() << job->error();
+                auto proxyError = translateImapError(job->error(), dynamic_cast<KIMAP2::LoginJob*>(job));
                 future.setError(proxyError, job->errorString());
             } else {
                 future.setFinished();
diff --git a/examples/imapresource/imapserverproxy.h b/examples/imapresource/imapserverproxy.h
index 9e73f68..7044a5e 100644
--- a/examples/imapresource/imapserverproxy.h
+++ b/examples/imapresource/imapserverproxy.h
@@ -31,6 +31,7 @@ namespace Imap {
 
 enum ErrorCode {
     NoError,
+    LoginFailed,
     HostNotFoundError,
     CouldNotConnectError,
     SslHandshakeError,