Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error with flickr API: Photoset not found #19

Closed
snarfed opened this issue May 28, 2015 · 6 comments
Closed

Error with flickr API: Photoset not found #19

snarfed opened this issue May 28, 2015 · 6 comments

Comments

@snarfed
Copy link

snarfed commented May 28, 2015

hi! thanks for writing flickr-set-get! i'm getting this error when i try to download a private photo set that i have a guest pass for. is that not supported?

$ flickr-set-get 72157653461216005 49022499@N00
ERROR: Error: Error with flickr API: Photoset not found (https://api.flickr.com/services/rest?photoset_id=72157653461216005&user_id=49022499%40N00&page=1&nojsoncallback=1&format=json&api_key=80ff7d163768e25928db697db1783e35&method=flickr.photosets.getPhotos)
@lmammino
Copy link
Owner

Hey @snarfed, glad you found flickr-set-get useful.
Unfortunately I haven't tested this case and I don't expect it to be supported.
It probably requires to use some kind of authentication mechanism or to send some password along with the API request.
I will open a specific issue with this case and I will try to get into the details as soon as I have some free time.

Sorry for the inconvenience.

@lmammino
Copy link
Owner

Hey @snarfed, I started to work on this issue some days ago, see dedicated branch and opened a dedicated issue: #20 .

I'll update you once it's finished and published on NPM (I hope before the end of this week).

Regards

@snarfed
Copy link
Author

snarfed commented Jun 15, 2015

that's great news, thank you!

@lmammino lmammino mentioned this issue Jun 19, 2015
7 tasks
@lmammino
Copy link
Owner

@snarfed just published (with a small delay 😅) the new version (1.1.0) that supports authenticated requests.
Please let me know if it works for you!

@snarfed
Copy link
Author

snarfed commented Jun 27, 2015

thanks so much for implementing this @lmammino!

@lmammino
Copy link
Owner

You are very welcome @snarfed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants