QDnsLookup: centralize printing of warnings

Any resolution error that is caused by an invalid request, invalid reply
(only happens on Unix), or a system error can be printed as a warning,
using category "qt.network.dnslookup". Those warnings are disabled by
default.

Change-Id: I5f7f427ded124479baa6fffd175fc40b3e21c969
Reviewed-by: Mårten Nordheim <marten.nordheim@qt.io>

src/network/kernel/qdnslookup.cpp

@@ -8,6 +8,7 @@
 #include <qapplicationstatic.h>
 #include <qcoreapplication.h>
 #include <qdatetime.h>
+#include <qloggingcategory.h>
 #include <qrandom.h>
 #include <qurl.h>
 
@@ -15,6 +16,8 @@
 
 QT_BEGIN_NAMESPACE
 
+static Q_LOGGING_CATEGORY(lcDnsLookup, "qt.network.dnslookup", QtCriticalMsg)
+
 namespace {
 struct QDnsLookupThreadPool : QThreadPool
 {
@@ -544,6 +547,7 @@ void QDnsLookup::lookup()
     d->isFinished = false;
     d->reply = QDnsLookupReply();
     if (!QCoreApplication::instance()) {
+        // NOT qCWarning because this isn't a result of the lookup
         qWarning("QDnsLookup requires a QCoreApplication");
         return;
     }
@@ -1057,20 +1061,47 @@ void QDnsLookupRunnable::run()
     if (qsizetype n = requestName.size(); n > MaxDomainNameLength || n == 0) {
         reply.error = QDnsLookup::InvalidRequestError;
         reply.errorString = QDnsLookup::tr("Invalid domain name");
-        emit finished(reply);
-        if (n)
-            qWarning("QDnsLookup: domain name being looked up is too long (%lld bytes)", n);
-        return;
+    } else {
+        // Perform request.
+        query(&reply);
+
+        // Sort results.
+        qt_qdnsmailexchangerecord_sort(reply.mailExchangeRecords);
+        qt_qdnsservicerecord_sort(reply.serviceRecords);
     }
 
-    // Perform request.
-    query(&reply);
-
-    // Sort results.
-    qt_qdnsmailexchangerecord_sort(reply.mailExchangeRecords);
-    qt_qdnsservicerecord_sort(reply.serviceRecords);
-
     emit finished(reply);
+
+    // maybe print the lookup error as warning
+    switch (reply.error) {
+    case QDnsLookup::NoError:
+    case QDnsLookup::OperationCancelledError:
+    case QDnsLookup::NotFoundError:
+    case QDnsLookup::ServerFailureError:
+    case QDnsLookup::ServerRefusedError:
+        break;      // no warning for these
+
+    case QDnsLookup::ResolverError:
+    case QDnsLookup::InvalidRequestError:
+    case QDnsLookup::InvalidReplyError:
+        qCWarning(lcDnsLookup()).nospace()
+                << "DNS lookup failed (" << reply.error << "): "
+              << qUtf16Printable(reply.errorString)
+              << "; request was " << this;  // continues below
+    }
+}
+
+inline QDebug operator<<(QDebug &d, QDnsLookupRunnable *r)
+{
+    // continued: print the information about the request
+    d << r->requestName.left(MaxDomainNameLength);
+    if (r->requestName.size() > MaxDomainNameLength)
+        d << "... (truncated)";
+    d << " type " << r->requestType;
+    if (!r->nameserver.isNull())
+        d << " to nameserver " << qUtf16Printable(r->nameserver.toString())
+          << " port " << (r->port ? r->port : DnsPort);
+    return d;
 }
 
 QT_END_NAMESPACE

src/network/kernel/qdnslookup_dummy.cpp

@@ -7,7 +7,6 @@ QT_BEGIN_NAMESPACE
 
 void QDnsLookupRunnable::query(QDnsLookupReply *reply)
 {
-    qWarning("Not yet supported on this OS");
     reply->error = QDnsLookup::ResolverError;
     reply->errorString = tr("Not yet supported on this OS");
     return;

src/network/kernel/qdnslookup_p.h

@@ -36,6 +36,7 @@ constexpr qsizetype MaxDomainNameLength = 255;
 constexpr quint16 DnsPort = 53;
 
 class QDnsLookupRunnable;
+QDebug operator<<(QDebug &, QDnsLookupRunnable *);
 
 class QDnsLookupReply
 {
@@ -179,6 +180,7 @@ private:
     QHostAddress nameserver;
     QDnsLookup::Type requestType;
     quint16 port;
+    friend QDebug operator<<(QDebug &, QDnsLookupRunnable *);
 };
 
 class QDnsRecordPrivate : public QSharedData

src/network/kernel/qdnslookup_unix.cpp

@@ -109,11 +109,9 @@ void QDnsLookupRunnable::query(QDnsLookupReply *reply)
     auto guard = qScopeGuard([&] { res_nclose(&state); });
 
     //Check if a nameserver was set. If so, use it
-    if (!applyNameServer(&state, nameserver, port)) {
-        qWarning("QDnsLookup: %s", "IPv6 nameservers are currently not supported on this OS");
+    if (!applyNameServer(&state, nameserver, port))
         return reply->setError(QDnsLookup::ResolverError,
                                QDnsLookup::tr("IPv6 nameservers are currently not supported on this OS"));
-    }
 #ifdef QDNSLOOKUP_DEBUG
     state.options |= RES_DEBUG;
 #endif