I'm working on a module to allow users to authenticate against their Google Apps account using OAuth, and want to use the OAuth Common module as a dependency. However, the current API isn't really usable for me. The getAccessToken() and other client methods make assumptions about the oauth paths, which aren't valid for Google's OAuth servers. Google Apps also requires additional non-standard parameters, which can't be specified with the current API - though the OAuth library itself can handle it.

I've attached a patch which adds this functionality. I hope you'll consider adding these features.

Cheers,
Stella

CommentFileSizeAuthor
oauth_common.patch3.6 KBstella

Comments

budda’s picture

budda
Primary language English
Location Cheshire
commented

Looks like the method updates in this patch are already implemented in the latest release of the project.

stella’s picture

stella commented
Status: Needs review » Fixed

Yep, looks like it was committed in http://drupal.org/cvs?commit=316266

Hugo Wetterberg’s picture

Hugo Wetterberg commented

Hi stella,
Yeah, it made it into the 1.x (sorry about not updating the issue status), but it isn't included in the 2.x release. What I've moved towards is instead to store endpoint information with the consumers themselves, so that the calling party doesn't have to know the specifics for the consumer to retrieve a authorization.

Status: Fixed » Closed (fixed)

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