PLATFORM-517: added timeouts

[n1te1337]

ping @rayward @dsolsona @bigcommerce-labs/platform

[rayward]

Don't think this PR is necessary, you can configure default request options (such as the timeouts) on the guzzle client itself.

http://docs.guzzlephp.org/en/latest/quickstart.html#creating-a-client

[n1te1337]

@rayward of course you can, but the question is whether the person using the client should know how to use/configure guzzle. If you follow this path you can just say that the whole client is unnecessary because all of the requests can be done with guzzle.

[rayward]

You'd expect them to know how to use it as it's a dependency of creating a Vault\Client.

If you follow this path you can just say that the whole client is unnecessary because all of the requests can be done with guzzle.

I wouldn't go that far, you're encapsulating how requests are sent and parsed specifically for vault.