diff options
author | Allan Sandfeld Jensen <allan.jensen@qt.io> | 2020-07-10 16:29:54 +0200 |
---|---|---|
committer | Allan Sandfeld Jensen <allan.jensen@qt.io> | 2020-07-15 14:48:00 +0200 |
commit | 9a90cd56c432f2ad3e811449cb199b11c6ed9de8 (patch) | |
tree | 9f1441e3c6508866ea77dcdc5138dd8bcbb2f4fa /tests/auto/core/qwebenginecookiestore | |
parent | 11a27178e42df66906dd12dcb5773ac6cb4426b8 (diff) |
Add the cookie filter to URL requests
Submodule src/3rdparty c91f4d20..809e16e4:
> Add cookie filter to URL requests cookie headers
> Use audio-manager app-name for MPRIS
Fixes: QTBUG-85526
Change-Id: Icab26cad3cea8e2ee021a3e589f41bf0543d64fa
Reviewed-by: Michael BrĂ¼ning <michael.bruning@qt.io>
Diffstat (limited to 'tests/auto/core/qwebenginecookiestore')
-rw-r--r-- | tests/auto/core/qwebenginecookiestore/qwebenginecookiestore.pro | 1 | ||||
-rw-r--r-- | tests/auto/core/qwebenginecookiestore/tst_qwebenginecookiestore.cpp | 72 |
2 files changed, 73 insertions, 0 deletions
diff --git a/tests/auto/core/qwebenginecookiestore/qwebenginecookiestore.pro b/tests/auto/core/qwebenginecookiestore/qwebenginecookiestore.pro index e99c7f493..9c239f1a7 100644 --- a/tests/auto/core/qwebenginecookiestore/qwebenginecookiestore.pro +++ b/tests/auto/core/qwebenginecookiestore/qwebenginecookiestore.pro @@ -1 +1,2 @@ include(../tests.pri) +include(../../shared/http.pri) diff --git a/tests/auto/core/qwebenginecookiestore/tst_qwebenginecookiestore.cpp b/tests/auto/core/qwebenginecookiestore/tst_qwebenginecookiestore.cpp index 2c41aa9b1..638b2e028 100644 --- a/tests/auto/core/qwebenginecookiestore/tst_qwebenginecookiestore.cpp +++ b/tests/auto/core/qwebenginecookiestore/tst_qwebenginecookiestore.cpp @@ -33,6 +33,9 @@ #include <QtWebEngineWidgets/qwebenginepage.h> #include <QtWebEngineWidgets/qwebengineprofile.h> +#include "httpserver.h" +#include "httpreqrep.h" + class tst_QWebEngineCookieStore : public QObject { Q_OBJECT @@ -56,6 +59,7 @@ private Q_SLOTS: void cookieSignals(); void batchCookieTasks(); void basicFilter(); + void basicFilterOverHTTP(); void html5featureFilter(); private: @@ -239,6 +243,74 @@ void tst_QWebEngineCookieStore::basicFilter() QCOMPARE(cookieAddedSpy.count(), 2); } +void tst_QWebEngineCookieStore::basicFilterOverHTTP() +{ + QWebEnginePage page(m_profile); + QWebEngineCookieStore *client = m_profile->cookieStore(); + + QAtomicInt accessTested = 0; + client->setCookieFilter([&](const QWebEngineCookieStore::FilterRequest &) { ++accessTested; return true; }); + + HttpServer httpServer; + + if (!httpServer.start()) + QSKIP("Failed to start http server"); + + QByteArray cookieRequestHeader; + connect(&httpServer, &HttpServer::newRequest, [&cookieRequestHeader](HttpReqRep *rr) { + if (rr->requestPath().size() <= 1) { + cookieRequestHeader = rr->requestHeader(QByteArrayLiteral("Cookie")); + rr->setResponseStatus(200); + if (cookieRequestHeader.isEmpty()) + rr->setResponseHeader(QByteArrayLiteral("Set-Cookie"), QByteArrayLiteral("Test=test")); + rr->sendResponse(); + } else { + rr->setResponseStatus(404); + rr->sendResponse(); + } + }); + + QSignalSpy loadSpy(&page, SIGNAL(loadFinished(bool))); + QSignalSpy cookieAddedSpy(client, SIGNAL(cookieAdded(const QNetworkCookie &))); + QSignalSpy cookieRemovedSpy(client, SIGNAL(cookieRemoved(const QNetworkCookie &))); + + page.load(httpServer.url()); + + QTRY_COMPARE_WITH_TIMEOUT(loadSpy.count(), 1, 30000); + QVERIFY(loadSpy.takeFirst().takeFirst().toBool()); + QTRY_COMPARE(cookieAddedSpy.count(), 1); + QTRY_COMPARE(accessTested.loadAcquire(), 3); + QVERIFY(cookieRequestHeader.isEmpty()); + + page.triggerAction(QWebEnginePage::Reload); + QTRY_COMPARE(loadSpy.count(), 1); + QVERIFY(loadSpy.takeFirst().takeFirst().toBool()); + QVERIFY(!cookieRequestHeader.isEmpty()); + QTRY_COMPARE(cookieAddedSpy.count(), 1); + QTRY_COMPARE(accessTested.loadAcquire(), 5); + + client->deleteAllCookies(); + QTRY_COMPARE(cookieRemovedSpy.count(), 1); + + client->setCookieFilter([&](const QWebEngineCookieStore::FilterRequest &) { ++accessTested; return false; }); + page.triggerAction(QWebEnginePage::ReloadAndBypassCache); + QTRY_COMPARE(loadSpy.count(), 1); + QVERIFY(loadSpy.takeFirst().takeFirst().toBool()); + QVERIFY(cookieRequestHeader.isEmpty()); + // Test cookies are NOT added: + QTest::qWait(100); + QCOMPARE(cookieAddedSpy.count(), 1); + QTRY_COMPARE(accessTested.loadAcquire(), 8); + + page.triggerAction(QWebEnginePage::Reload); + QTRY_COMPARE(loadSpy.count(), 1); + QVERIFY(loadSpy.takeFirst().takeFirst().toBool()); + QVERIFY(cookieRequestHeader.isEmpty()); + QCOMPARE(cookieAddedSpy.count(), 1); + + (void) httpServer.stop(); +} + void tst_QWebEngineCookieStore::html5featureFilter() { QWebEnginePage page(m_profile); |