airbnb / interferon

Signaling you about infrastructure or application issues
MIT License
239 stars 44 forks source link

[dogapi] Allow configurable api timeout #12

Closed Jason-Jian closed 8 years ago

Jason-Jian commented 8 years ago

The default dogapi timeout 5s isn't always enough. Increasing it to 15s while making it configurable

15s was a magic number I chose, let me know if it should be more conservatively increased.

@igor47 cc: @wyao @airbnb/product-infra

igor47 commented 8 years ago

done with pass, ping me when it's ready for another look.

Jason-Jian commented 8 years ago

@igor47 can you take a 2nd look?

igor47 commented 8 years ago

LGTM!

Jason-Jian commented 8 years ago

thx!