path: root/src/network/access/qhttpthreaddelegate.cpp
diff options
authorMårten Nordheim <>2021-11-26 13:15:46 +0100
committerMårten Nordheim <>2021-11-30 00:04:38 +0100
commit4c930e9d13738ac0faaaed3aa5b1c34509b04019 (patch)
tree29b1b1f41ebd2991e7fcece6b99aa820f427845e /src/network/access/qhttpthreaddelegate.cpp
parentca9d72f4652e21ecaf6e50be2e7b851c4ec69486 (diff)
QNAM: Disable h2c by default
And since it's relatively unlikely to be used, just leave it behind a environment variable for now. [ChangeLog][QtNetwork][Potentially Source-Incompatible] Support for clear-text http/2 was disabled due to incompatibility with certain servers. If you were relying on this feature you must re-enable it by setting the QT_NETWORK_ALLOW_H2C environment variable. For a later version of Qt it will get a dedicated attribute. Pick-to: 6.2 Task-number: QTBUG-98642 Change-Id: Id3e360726e285b3128e3e3f4bce9440404c9ad6e Reviewed-by: Qt CI Bot <> Reviewed-by: Maximilian Goldstein <> Reviewed-by: Timur Pocheptsov <>
Diffstat (limited to 'src/network/access/qhttpthreaddelegate.cpp')
1 files changed, 6 insertions, 0 deletions
diff --git a/src/network/access/qhttpthreaddelegate.cpp b/src/network/access/qhttpthreaddelegate.cpp
index 50a14b6258..243492d466 100644
--- a/src/network/access/qhttpthreaddelegate.cpp
+++ b/src/network/access/qhttpthreaddelegate.cpp
@@ -290,6 +290,12 @@ void QHttpThreadDelegate::startRequest()
connectionType = QHttpNetworkConnection::ConnectionTypeHTTP2Direct;
+ // Use HTTP/1.1 if h2c is not allowed and we would otherwise choose to use it
+ if (!ssl && connectionType == QHttpNetworkConnection::ConnectionTypeHTTP2
+ && !httpRequest.isH2cAllowed()) {
+ connectionType = QHttpNetworkConnection::ConnectionTypeHTTP;
+ }
#if QT_CONFIG(ssl)
// See qnetworkreplyhttpimpl, delegate's initialization code.
Q_ASSERT(!ssl ||;