diff options
author | Mårten Nordheim <marten.nordheim@qt.io> | 2020-09-10 16:30:36 +0200 |
---|---|---|
committer | Mårten Nordheim <marten.nordheim@qt.io> | 2020-09-11 11:30:57 +0200 |
commit | 5a47939d5c07a968f27562a3ebc800fcc2b225bc (patch) | |
tree | a7cf1c3aef4d47a164230fda9e7d9e61505a5db7 /tests | |
parent | 7dd99cd4885479feb7e09c55ab118c5ffb5b8476 (diff) |
tst_QNetworkReply: Avoid race in ioGetFromHttpWithAuth
Our authentication code is race-y by design:
1. When two requests are fired off and queued at the same time in the
same QHttpNetworkConnection then if one of them encounters
"authentication required" then it will copy whatever credentials it got
to all the other channels in the connection. This is likely what the
first part of the test is testing.
2. If a later request is fired off and it includes credentials in the
url then the newly included credentials should be used instead of the
cached ones.
The race here can occurr when one socket either takes too long to
connect or the connected signal is not received early enough. Then the
first socket is used for both requests and then we can hit case #2 when
the url contains credentials.
Pick-to: 5.15
Change-Id: I646a5378d8c1256b2de98b51912953df29f68cb2
Reviewed-by: Timur Pocheptsov <timur.pocheptsov@qt.io>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/auto/network/access/qnetworkreply/tst_qnetworkreply.cpp | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/tests/auto/network/access/qnetworkreply/tst_qnetworkreply.cpp b/tests/auto/network/access/qnetworkreply/tst_qnetworkreply.cpp index 98f468e49e..fa574b8d92 100644 --- a/tests/auto/network/access/qnetworkreply/tst_qnetworkreply.cpp +++ b/tests/auto/network/access/qnetworkreply/tst_qnetworkreply.cpp @@ -3499,7 +3499,11 @@ void tst_QNetworkReply::ioGetFromHttpWithAuth() QNetworkRequest request(url); { QNetworkReplyPtr reply1(manager.get(request)); - QNetworkReplyPtr reply2(manager.get(request)); + QUrl copy = url; + copy.setUserName(QString()); + copy.setPassword(QString()); + QNetworkRequest request2(copy); + QNetworkReplyPtr reply2(manager.get(request2)); DataReader reader1(reply1); DataReader reader2(reply2); QSignalSpy finishedspy(reply1.data(), SIGNAL(finished())); |