Closed (fixed)
Project:
Flickr API
Version:
7.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
30 Jan 2012 at 06:26 UTC
Updated:
21 Apr 2012 at 09:41 UTC
Jump to comment: Most recent file
Flickr is switching to OAuth exclusively after July 31, 2012. It currently supports its own authentication in parallel with OAuth. So this module should transition to using OAuth before July 31.
It seems reasonably straightforward. I'm happy to take this on if I find some time. For the moment, it's up for the taking. If anyone else codes it up, please be aware of some other unintegrated auth-related patches I've made recently which have not been reviewed. Quiet little module, this one.
This issue applies to all versions. Seems to be no way to indicate that elegantly.
Comments
Comment #0.0
hughbris commentedBorked link markup, sorry
Comment #1
BarisW commentedHi Hugh,
thanks for posting this. Did you happen to be working on this? If not, I'll have a look.
I've just rolled a new 7.x-1.2 release; please work on the current dev as I've implemented the 3.1 version of the Flickr API as well.
Thanks a lot!
Comment #2
BarisW commentedMay I suggest to not re-invent the wheel but use the OAuth module as dependency and only write integration with that module? If anyone knows how to implement it; please do.
Because FlickrAPI 3.1 (the latest version) has no implementation with OAuth authentication, I think we need to override the current class to change the auth() function. Like this:
As you can see, the hardest part (
<?php // Add OAuth integration here. ?>) still has to be taken care of.Comment #3
BarisW commentedI managed to integrate the OAuth module and implement the authorization flow in the FlickAPI admin settings page. The only thing left now is to change the way Flickr is called (http://www.flickr.com/services/api/auth.oauth.html#call_api).
Will be continued.
Comment #4
BarisW commentedCommitted.
Comment #5
BarisW commentedI even added some nice icons ;)
Comment #6.0
(not verified) commentedBorked apostrophe, sorry :%