Skip Menu |

This queue is for tickets about the Catalyst-Authentication-Credential-OAuth CPAN distribution.

Report information
The Basics
Id: 50383
Status: resolved
Priority: 0/
Queue: Catalyst-Authentication-Credential-OAuth

People
Owner: Nobody in particular
Requestors: CWW [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 0.02
Fixed in: (no value)



Subject: Package does not support new "verifier" requirement for "access token"
This module does not work with Net::OAuth v0.19 (the latest) because Net::OAuth now requires the "verifier" parameter (which is just the "oauth_verifier" we got from the previous request) to be sent along with the "access token" call. I've attached a patch that fixes the problem for me.
Subject: catalyst-oauth.patch
--- OAuth.pm-bak 2009-10-10 17:01:48.448575152 -0700 +++ OAuth.pm 2009-10-10 17:02:43.274447435 -0700 @@ -72,6 +72,7 @@ sub authenticate { token => $response->token, token_secret => '', request_url => $provider->{access_token_endpoint}, + verifier => $c->req->params->{oauth_verifier}, ); $request->sign;
Also, FWIW, here's the error I would get before the patch. [error] Caught exception in Foo::Controller::Public->twitter_authorize "Missing required parameter 'verifier' at /home/cww/local/perl5lib/lib/Net/OAuth/Message.pm line 91."
This has been added to version 0.03, just released. Thanks for the bug report and patch!