Funkwhale/Clementine-audio-player-Mac-Linux-Windows
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

oauth: Fix redirect

Browse Source 
Query params were not added to redirect url prior to inclusion in oauth
request. This change fixes OneDrive authentication.

Regression introduced in 8b226c2171
This commit is contained in:
Jim Broadus 2021-02-21 00:06:13 -08:00 committed by John Maguire
parent 1a3828e2c1
commit b40d9ed44b
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/internet/core/oauthenticator.cpp
View File

@ -70,13 +70,14 @@ void OAuthenticator::StartAuthorisation(const QString& oauth_endpoint,
redirect_url = server->url();
}
redirect_url.setQuery(redirect_url_query);
url_query.addQueryItem("redirect_uri", redirect_url.toString());
if (!scope.isEmpty()) { // Empty scope is valid for Dropbox.
url_query.addQueryItem("scope", scope);
}
url.setQuery(url_query);
redirect_url.setQuery(redirect_url_query);
NewClosure(server, SIGNAL(Finished()), this, &OAuthenticator::RedirectArrived,
server, redirect_url);