Gem release check is failing with HTTP timeout about smart_todo

[rafaelfranca]

Thank you for the issue. Are you able to open a PR adding some configuration to the timeout of the client? I'd like to avoid adding faraday as a dependency of the gem since I don't think it is necessary just yet.

[rzaharenkov]

Sure, I can do it. Btw, I have just found there is similar code for slack API but it actually has timeouts:

https://github.com/Shopify/smart_todo/blob/master/lib/smart_todo/slack_client.rb#L30-L31

Do you think we have to use different configs for rubygems/slack or just provide generic HTTP client options? It could be handy to configure it separately but I think in the most of cases consumer of this code doesn't care. So I'd keep it simple with single config.

For the options itself we also have few options:

• configure options as Hash;
• allow to configure a lambda that should be executed after client is built;
• allow to configure a lambda that should accept the URL and build an instance of Net::HTTP;

I'd prefer to keep it simple and use a Hash.

[rafaelfranca]

Single option as a Hash is a good start to me. If we need to build something more complicated users will ask for it.