Http: Improve error string when there is no reason phrase

The reason phrase is optional, and in http2+ it is not included at all.
So when we generate the error string as 'server replied: %2' it ends
up being unnatural and ugly, and certainly not fit for display.

To improve that, whenever there is no reason phrase, we instead
generate the string like 'server replied with status code %2'
and output the status code instead, which is included.

Fixes: QTBUG-140126
Pick-to: 6.10 6.9 6.8
Change-Id: Ife3ed920f6e1a15d83e5cd5dc9b0568fb195b642
Reviewed-by: Timur Pocheptsov <[email protected]>
Reviewed-by: Mate Barany <[email protected]>

1 files changed, 24 insertions, 11 deletions

diff --git a/src/network/access/qhttpthreaddelegate.cpp b/src/network/access/qhttpthreaddelegate.cpp
index 45a5c167265..fbbc55dc4a4 100644
--- a/src/network/access/qhttpthreaddelegate.cpp
+++ b/src/network/access/qhttpthreaddelegate.cpp
@@ -488,6 +488,21 @@ void QHttpThreadDelegate::readyReadSlot()
     }
 }
 
+static QString makeServerErrorString(int code, const QUrl &url, const QString &reasonPhrase)
+{
+    QString msg;
+    if (!reasonPhrase.isEmpty()) {
+        msg = QLatin1StringView(QT_TRANSLATE_NOOP("QNetworkReply",
+                                                  "Error transferring %1 - server replied: %2"))
+                      .arg(url.toString(), reasonPhrase);
+    } else {
+        msg = QLatin1StringView(QT_TRANSLATE_NOOP("QNetworkReply",
+                                                  "Error transferring %1 - server replied with status code %2"))
+                      .arg(url.toString(), QString::number(code));
+    }
+    return msg;
+}
+
 void QHttpThreadDelegate::finishedSlot()
 {
     if (!httpReply)
@@ -509,12 +524,11 @@ void QHttpThreadDelegate::finishedSlot()
 #endif
 
     if (httpReply->statusCode() >= 400) {
-            // it's an error reply
-            QString msg = QLatin1StringView(QT_TRANSLATE_NOOP("QNetworkReply",
-                                                              "Error transferring %1 - server replied: %2"));
-            msg = msg.arg(httpRequest.url().toString(), httpReply->reasonPhrase());
-            emit error(statusCodeFromHttp(httpReply->statusCode(), httpRequest.url()), msg);
-        }
+        // it's an error reply
+        QString msg = makeServerErrorString(httpReply->statusCode(), httpRequest.url(),
+                                            httpReply->reasonPhrase());
+        emit error(statusCodeFromHttp(httpReply->statusCode(), httpRequest.url()), msg);
+    }
 
     if (httpRequest.isFollowRedirects() && httpReply->isRedirecting())
         emit redirected(httpReply->redirectUrl(), httpReply->statusCode(), httpReply->request().redirectCount() - 1);
@@ -535,11 +549,10 @@ void QHttpThreadDelegate::synchronousFinishedSlot()
     qDebug() << "QHttpThreadDelegate::synchronousFinishedSlot() thread=" << QThread::currentThreadId() << "result=" << httpReply->statusCode();
 #endif
     if (httpReply->statusCode() >= 400) {
-            // it's an error reply
-            QString msg = QLatin1StringView(QT_TRANSLATE_NOOP("QNetworkReply",
-                                                              "Error transferring %1 - server replied: %2"));
-            incomingErrorDetail = msg.arg(httpRequest.url().toString(), httpReply->reasonPhrase());
-            incomingErrorCode = statusCodeFromHttp(httpReply->statusCode(), httpRequest.url());
+        // it's an error reply
+        incomingErrorDetail = makeServerErrorString(httpReply->statusCode(), httpRequest.url(),
+                                                    httpReply->reasonPhrase());
+        incomingErrorCode = statusCodeFromHttp(httpReply->statusCode(), httpRequest.url());
     }
 
     isCompressed = httpReply->isCompressed();