From 31dbe72afb5e04cd86d4f107e7257fb60aaa087b Mon Sep 17 00:00:00 2001
From: Christian Mollekopf <chrigi_1@fastmail.fm>
Date: Thu, 25 Jan 2018 14:43:07 +0100
Subject: Do the logging in the resource code.

---
 examples/mailtransportresource/mailtransport.cpp | 18 +++++-------------
 1 file changed, 5 insertions(+), 13 deletions(-)

(limited to 'examples/mailtransportresource/mailtransport.cpp')

diff --git a/examples/mailtransportresource/mailtransport.cpp b/examples/mailtransportresource/mailtransport.cpp
index afe0257..65299a2 100644
--- a/examples/mailtransportresource/mailtransport.cpp
+++ b/examples/mailtransportresource/mailtransport.cpp
@@ -21,7 +21,6 @@
 #include <QByteArray>
 #include <QList>
 #include <QDebug>
-#include <common/log.h>
 
 extern "C" {
 
@@ -142,13 +141,14 @@ bool sendMessageCurl(const char *to[], int numTos, const char *cc[], int numCcs,
 
         res = curl_easy_perform(curl);
         if(res != CURLE_OK) {
+            errorMessage += "Error code: " + QByteArray::number(res) + ", ";
             errorMessage += curl_easy_strerror(res);
             errorMessage += "; ";
         }
         long http_code = 0;
         curl_easy_getinfo (curl, CURLINFO_RESPONSE_CODE, &http_code);
         if (http_code == 200 && res != CURLE_ABORTED_BY_CALLBACK) {
-                //Succeeded
+            //Succeeded
         } else {
             errorMessage += errorBuffer;
         }
@@ -161,10 +161,8 @@ bool sendMessageCurl(const char *to[], int numTos, const char *cc[], int numCcs,
 
 };
 
-bool MailTransport::sendMessage(const KMime::Message::Ptr &message, const QByteArray &server, const QByteArray &username, const QByteArray &password, const QByteArray &cacert, MailTransport::Options options)
+MailTransport::SendResult MailTransport::sendMessage(const KMime::Message::Ptr &message, const QByteArray &server, const QByteArray &username, const QByteArray &password, const QByteArray &cacert, MailTransport::Options options)
 {
-    QByteArray msg = message->encodedContent();
-
     QByteArray from(message->from(true)->mailboxes().isEmpty() ? QByteArray() : message->from(true)->mailboxes().first().address());
     QList<QByteArray> toList;
     for (const auto &mb : message->to(true)->mailboxes()) {
@@ -177,9 +175,6 @@ bool MailTransport::sendMessage(const KMime::Message::Ptr &message, const QByteA
     const bool verifyPeer = options.testFlag(VerifyPeers);
     const bool useTls = options.testFlag(UseTls);
 
-    SinkLog() << "Sending message " << server << username << password << "CaCert: " << cacert  << "Use tls: " << useTls << " Verify peer: " << verifyPeer;
-    SinkTrace() << "Sending message " << msg;
-
     const int numTos = toList.size();
     const char* to[numTos];
     for (int i = 0; i < numTos; i++) {
@@ -196,9 +191,6 @@ bool MailTransport::sendMessage(const KMime::Message::Ptr &message, const QByteA
     serverAddress.replace("smtps://", "smtp://");
 
     QByteArray errorMessage;
-    auto ret =  sendMessageCurl(to, numTos, cc, numCcs, msg, useTls, from.isEmpty() ? nullptr : from, username, password, serverAddress, verifyPeer, cacert, errorMessage);
-    if (!ret) {
-        SinkWarning() << "Failed to send message: " << errorMessage;
-    }
-    return ret;
+    auto ret = sendMessageCurl(to, numTos, cc, numCcs, message->encodedContent(), useTls, from.isEmpty() ? nullptr : from, username, password, serverAddress, verifyPeer, cacert, errorMessage);
+    return {ret, errorMessage};
 }
-- 
cgit v1.2.3