Active
Project:
OAuth 1.0
Version:
6.x-3.0-beta3
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
19 Apr 2010 at 13:32 UTC
Updated:
22 Mar 2012 at 21:44 UTC
In oauth_common.pages.inc, the function _oauth_common_authorize() returns a form with two links to deny access to an application which requested authentication via oauth.
$form['warning'] = array(
'#type' => 'item',
'#value' => l(t('If you don\'t know what !appname is, or don\'t want to give it access to your content, just click here and we\'ll take you away from this page without granting !appname any access to !sitename.', $msgargs), 'oauth/authorization/deny/' . $token->key),
'#attributes' => array(
'class' => 'abort-authorization',
),
);
and
$form['deny'] = array(
'#type' => 'item',
'#value' => l(t('Deny access'), 'oauth/authorization/deny/' . $token->key),
'#attributes' => array(
'class' => 'deny-access',
),
);
The menu callback has not been implemented in oauth_common_menu(). Perhaps it has been implemented somewhere else, so forgive my ignorance.
I thought to post this bug already, before trying to solve it.
Comments
Comment #1
Hugo Wetterberg commentedComment #2
voxpelli commentedChanging this to critical as it's not good at all for two such vital links to be broken.
Comment #3
voxpelli commentedReally really really need to fix this prior to stable release.
Comment #4
paulmckibbenSubscribe. This functionality is vital for me.
Comment #5
voxpelli commentedWon't have time for this - deassigning myself to not give a false impression.