Falkon/falkonOfficial
Falkon/falkonOfficial
1
Fork 0
mirror of https://invent.kde.org/network/falkon.git synced 2024-11-11 01:22:10 +01:00
Code Releases Activity

LocationBar: Fix resolving correct LoadAction when url contains %20

Browse Source 
BUG: 392445
FIXED-IN: 3.0.1
This commit is contained in:
David Rosca 2018-03-28 18:54:26 +02:00
parent 7595cef9ee
commit 2f6cd7a559
No known key found for this signature in database
GPG Key ID: EBC3FC294452C6D8
3 changed files with 16 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
autotests/locationbartest.cpp
View File

@ -214,4 +214,17 @@ void LocationBarTest::loadAction_issue2578()
QCOMPARE(action.type, LocationBar::LoadAction::Invalid); QCOMPARE(action.type, LocationBar::LoadAction::Invalid);
} }
void LocationBarTest::loadAction_kdebug392445()
{
// %20 in url will make it incorrectly treat as web search
qzSettings->searchFromAddressBar = true;
LocationBar::LoadAction action;
action = LocationBar::loadAction("http://www.example.com/my%20beautiful%20page");
QCOMPARE(action.type, LocationBar::LoadAction::Url);
QCOMPARE(action.loadRequest.url(), QUrl("http://www.example.com/my%20beautiful%20page"));
}
FALKONTEST_MAIN(LocationBarTest) FALKONTEST_MAIN(LocationBarTest)

1
autotests/locationbartest.h
View File

@ -34,4 +34,5 @@ private Q_SLOTS:
void loadAction_kdebug389491(); void loadAction_kdebug389491();
void loadActionSpecialSchemesTest(); void loadActionSpecialSchemesTest();
void loadAction_issue2578(); void loadAction_issue2578();
void loadAction_kdebug392445();
}; };

4
src/lib/navigation/locationbar.cpp
View File

@ -277,12 +277,12 @@ LocationBar::LoadAction LocationBar::loadAction(const QString &text)
} }
// Otherwise load as url // Otherwise load as url
const QUrl &guessedUrl = QUrl::fromUserInput(t); const QUrl guessedUrl = QUrl::fromUserInput(t);
if (guessedUrl.isValid()) { if (guessedUrl.isValid()) {
// Always allow javascript: to be loaded // Always allow javascript: to be loaded
const bool forceLoad = guessedUrl.scheme() == QL1S("javascript"); const bool forceLoad = guessedUrl.scheme() == QL1S("javascript");
// Only allow spaces in query // Only allow spaces in query
if (forceLoad || !QzTools::containsSpace(guessedUrl.toString(QUrl::RemoveQuery))) { if (forceLoad || !QzTools::containsSpace(t) || !QzTools::containsSpace(guessedUrl.toString(QUrl::RemoveQuery))) {
// Only allow whitelisted schemes // Only allow whitelisted schemes
static const QSet<QString> whitelistedSchemes = { static const QSet<QString> whitelistedSchemes = {
QSL("http"), QSL("https"), QSL("ftp"), QSL("file"), QSL("http"), QSL("https"), QSL("ftp"), QSL("file"),