tobias/friendica
1
0
Fork 0
Code Pull requests Activity

Use strtok to find QP's in Uri for OAuth redirect processing

Browse source
This commit is contained in:
Hank Grabowski 2023-11-06 11:25:13 -05:00
parent 89e105afbe
commit 35a726a961
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/Module/OAuth/Token.php
View file

@ -90,10 +90,10 @@ class Token extends BaseApi
$me = null; $me = null;
} elseif ($request['grant_type'] == 'authorization_code') { } elseif ($request['grant_type'] == 'authorization_code') {
// For security reasons only allow freshly created tokens // For security reasons only allow freshly created tokens
$uri = new Uri($request['redirect_uri']); $redirect_uri = strtok($request['redirect_uri'],'?');
$condition = [ $condition = [
"`redirect_uri` LIKE ? AND `id` = ? AND `code` = ? AND `created_at` > ?", "`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); $token = DBA::selectFirst('application-view', ['access_token', 'created_at', 'uid'], $condition);

4
src/Security/OAuth.php
View file

@ -131,8 +131,7 @@ class OAuth
} }
if (!empty($redirect_uri)) { if (!empty($redirect_uri)) {
$uri = new Uri($redirect_uri); $redirect_uri = strtok($redirect_uri, '?');
$redirect_uri = $uri->getScheme() . '://' . $uri->getHost() . $uri->getPath();
$condition = DBA::mergeConditions($condition, ["`redirect_uri` LIKE ?", '%' . $redirect_uri . '%']); $condition = DBA::mergeConditions($condition, ["`redirect_uri` LIKE ?", '%' . $redirect_uri . '%']);
} }
@ -143,6 +142,7 @@ class OAuth
} }
// The redirect_uri could contain several URI that are separated by spaces. // The redirect_uri could contain several URI that are separated by spaces.
$exploded = explode(' ', $application['redirect_uri']);
if (($application['redirect_uri'] != $redirect_uri) && !in_array($redirect_uri, explode(' ', $application['redirect_uri']))) { if (($application['redirect_uri'] != $redirect_uri) && !in_array($redirect_uri, explode(' ', $application['redirect_uri']))) {
return []; return [];
} }