Merge pull request #13614 from HankG/oauth-redirect-fix

Fix OAuth Redirect Url Scrubbing
This commit is contained in:
Hypolite Petovan 2023-11-06 10:12:55 -08:00 committed by GitHub
commit 9d93eaf810
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 4 deletions

View file

@ -90,10 +90,10 @@ class Token extends BaseApi
$me = null;
} elseif ($request['grant_type'] == 'authorization_code') {
// For security reasons only allow freshly created tokens
$uri = new Uri($request['redirect_uri']);
$redirect_uri = strtok($request['redirect_uri'],'?');
$condition = [
"`redirect_uri` LIKE ? AND `id` = ? AND `code` = ? AND `created_at` > ?",
'%' . $uri->getScheme() . '://' . $uri->getHost() . $uri->getPath() . '%', $application['id'], $request['code'], DateTimeFormat::utc('now - 5 minutes')
$redirect_uri, $application['id'], $request['code'], DateTimeFormat::utc('now - 5 minutes')
];
$token = DBA::selectFirst('application-view', ['access_token', 'created_at', 'uid'], $condition);

View file

@ -131,8 +131,7 @@ class OAuth
}
if (!empty($redirect_uri)) {
$uri = new Uri($redirect_uri);
$redirect_uri = $uri->getScheme() . '://' . $uri->getHost() . $uri->getPath();
$redirect_uri = strtok($redirect_uri, '?');
$condition = DBA::mergeConditions($condition, ["`redirect_uri` LIKE ?", '%' . $redirect_uri . '%']);
}