API Down Bug #12

Closed
wants to merge 1 commit into
from

2 participants

@jrozner

I've noticed that the tumblr api is down a lot and when it is the response returned in nil which causes a NoMethodError because it's trying to access an array element on nil in the count method of Post. I've added a resque block to return 0 when tumblr api is down and a response isn't returned so that an exception isn't thrown.

@jeffkreeftmeijer

Hey Joe,

Thanks for the pull request, I'm sorry for taking so long to respond. I haven't been working on the Tumblr gem for a long time (I'm not even using Tumblr anymore right now) and I'm still looking for a new maintainer. Because of this, it's difficult to look into patches because they might break something. With your patch, this probably isn't the case, but can you add a simple test for this -- to make sure it won't regress -- and make sure everything still works?

Thanks again!

  • Jeff
@jrozner

Sure no problem. I'm not planning on using Tumblr again in the future either after the issues I've had with it and the fact that I can more quickly throw together a simple blogging system in rails than it took to integrate. It'll have to wait until next week because I'm busy with finals right now but I'll throw in a couple tests and modify the pull request.

@jrozner jrozner closed this Nov 22, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment