As mentioned on drupal_goto(): // Do not redirect to an absolute URL originating from user input.

This means when you pass on an absolute url, it's not used. _oauth_common_authorize() has to be changed:

    $query = $_GET;
    unset($query['q']); // why are there so few q's?
    drupal_goto('user/login', array(
      'destination' => url('oauth/authorize', array(
        'query' => $query,
        'absolute' => TRUE,
      )),
    ));

Find patch attached for the fix.

CommentFileSizeAuthor
oauth_common.pages_.inc-absolute.patch603 bytestoemaz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Hugo Wetterberg’s picture

Project: OAuth Common (deprecated) » OAuth 1.0
Version: 6.x-1.0-beta5 » 6.x-3.0-beta1
voxpelli’s picture

Status: Active » Needs review
voxpelli’s picture

Version: 6.x-3.0-beta1 » 6.x-3.0-beta2
Status: Needs review » Fixed

Fixed - thanks for the patch and sorry for the delay!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.