Revised Flickr add–on for Known using OAuth 1.0a

idno / flickr

Flickr synchronization plugin for Known

http://withknown.com/

Other

6 stars 7 forks source link

Closed cdn closed 5 years ago

cdn commented 6 years ago

Replaces external library, so not quite an answer to #13

Will probably need more revision before merge

Likely fixes #4 #11

cdn commented 6 years ago

I don’t known whether SmugMug’s take over of Flickr will have any API implications.

mapkyca commented 5 years ago

Hmm... I've merged and tried this, but I'm getting

"Invalid auth/bad request (got a 411, expected HTTP/1.1 20X or a redirect)"

..on connect.

Any thoughts?

mapkyca commented 5 years ago

Had to revert, but please feel free to re-submit

cdn commented 5 years ago

411 occurs back on version 0.9.9 2018032701 backed by PostgreSQL

It didn’t occur on version 0.9.9 2017051501 backed by SQLite.

Today, having disconnected the SQLite blog: gets through to Flickr (https://www.flickr.com/services/oauth/authorize?oauth_token=[hexadecimal–hyphenated string]&perms=write) (when oauth module is actually installed).

I don’t know why there’s an HTTP 411 Length error [yet].

cdn commented 5 years ago

Addendum: SQLite–backed blog is on php5 FPM PostgreSQL–backed blog is on php7.0 FPM

Line 35 is the call to the php oauth library’s getRequestToken method, perhaps that has changed between PHP versions?

mapkyca commented 5 years ago

If it helps this was PHP 7 with a mongo back end.

cdn commented 5 years ago

PostgreSQL with PHP5 still works.

PostgreSQL with PHP7.0 works if I add $callback as the second parameter, per the 8–year old note: http://php.net/manual/en/oauth.getrequesttoken.php

mapkyca commented 5 years ago

Cool! Could you update and resubmit the PR?