diff options
| author | Tim Ruffing <[email protected]> | 2018-10-05 10:24:52 +0200 |
|---|---|---|
| committer | Tim Ruffing <[email protected]> | 2018-10-09 11:54:05 +0200 |
| commit | 7d173c4cd1885ff5bcf9e5f8f7f712138dd8a445 (patch) | |
| tree | c997b0f0319a64d2233e0246c5a89b1631c86b8c /src/httpserver.cpp | |
| parent | Merge #14395: Fix typos and cleanup (diff) | |
| download | discoin-7d173c4cd1885ff5bcf9e5f8f7f712138dd8a445.tar.xz discoin-7d173c4cd1885ff5bcf9e5f8f7f712138dd8a445.zip | |
qt: Revert "Force TLS1.0+ for SSL connections"
This reverts commit 15e26a6a9afe299b9ca6fced73b876644365879b, whose
purpose was to tweak the Qt configuration to force TLS, i.e., to
disable SSLv3, in Qt versions >= 5.5. However, the default behavior
of Qt >= 5.4 is to disable SSLv3 anyway [1], so the configuration
tweak is redundant.
With Qt 5.11.2, the configuration tweak is not only redundant but in
fact provokes a deadlock (#14359) due to Qt 5.11.2 being incompatible
with OpenSSL 1.1.1 [2]. Since the deadlock occurs at the early startup
stage of bitcoin-qt, it renders bitcoin-qt entirely non-functional
when compiled against OpenSSL 1.1.1 and Qt 5.11.2 (and possible future
combinations of OpenSSL and Qt versions).
This commit fixes #14359 by removing the redundant code.
[1] https://code.qt.io/cgit/qt/qtbase.git/commit/?id=3fd2d9eff8c1f948306ee5fbfe364ccded1c4b84
[2] https://bugreports.qt.io/browse/QTBUG-70956
Diffstat (limited to 'src/httpserver.cpp')
0 files changed, 0 insertions, 0 deletions