Add more logging

Author: erikjv

src/libsync/httplogger.cpp

@@ -144,21 +144,26 @@ void HttpLogger::logRequest(QNetworkReply *reply, QNetworkAccessManager::Operati
         return;
     }
 
+    const auto logError = [reply, operation]() {
+        auto url = reply->request().url();
+        qCInfo(lcNetworkHttp).noquote().nospace() << "An error occurred for " << url.toDisplayString() << ": " << reply->errorString() << " (" << reply->error()
+                                                  << ", " << operation << ")";
+    };
+
     auto ctx = std::make_unique<HttpContext>(reply->request());
 
     // device should still exist, lets still use a qpointer to ensure we have valid data
-    const auto logSend = [ctx = ctx.get(), operation, reply, device = QPointer<QIODevice>(device), deviceRaw = device](bool cached = false) {
+    const auto logSend = [ctx = ctx.get(), operation, reply, device = QPointer<QIODevice>(device), deviceRaw = device, logError](bool cached = false) {
         Q_ASSERT(!deviceRaw || device);
         if (!ctx->send) {
             ctx->send = true;
             ctx->timer.reset();
-        } else {
+        } else if (ctx->lastUrl != reply->url()) {
             // this is a redirect
-            if (ctx->lastUrl != reply->url()) {
-                ctx->addRedirect(reply->url());
-            } else {
-                Q_UNREACHABLE();
-            }
+            ctx->addRedirect(reply->url());
+        } else {
+            // Probably an error (time-out?).
+            logError();
         }
 
         const auto request = reply->request();
@@ -169,6 +174,7 @@ void HttpLogger::logRequest(QNetworkReply *reply, QNetworkAccessManager::Operati
         logHttp(requestVerb(operation, request), ctx, std::move(header), device, cached);
     };
     QObject::connect(reply, &QNetworkReply::requestSent, reply, logSend, Qt::DirectConnection);
+    QObject::connect(reply, &QNetworkReply::errorOccurred, reply, logError, Qt::DirectConnection);
 
 
     QObject::connect(

src/libsync/syncengine.cpp

@@ -819,7 +819,7 @@ void SyncEngine::abort(const QString &reason)
         disconnect(_discoveryPhase.get(), nullptr, this, nullptr);
     }
     if (aborting) {
-        qCInfo(lcEngine) << "Aborting sync";
+        qCInfo(lcEngine) << "Aborting sync, stated reason:" << reason;
         if (!_goingDown) {
             Q_EMIT syncError(tr("Aborted due to %1").arg(reason));
         }