Allow cancelable requests using contexts

[dadgar]

It would be nice to provide a mechanism in which to be able to cancel the request. If we standardized on using contexts, the cancel could be done because it is no longer needed or to implement a timeout (https://golang.org/pkg/context/#WithTimeout)

[mitchellh]

Yeah I think standardizing on context.Context makes sense. My recommendations would be:

• Make it a field in Client

• Modify (break API, assume vendoring) the interfaces Getter, Decompressor, etc. to support a first context parameter. Those should be honored for cancellation if possible. I think breaking API compat at this layer is safer than the top-level, since its generally unlikely anyone is using a custom Getter.

• Add new top-level GetContext, GetFileContext, GetAnyContext methods so we preserve backwards compatibility on the top-level funcs.

[MithunArunan]

I will pick this up. Can you please assign it to me @dadgar @mitchellh

[dadgar]

@MithunArunan Feel free to post a PR once you have the feature. We do not generally assign tickets to outside contributors.

[azr]

Howdy 🙂 ! The go-getter is now cancellable as per #133 I think this issue can be closed !

Feel free to open an issue is there's a missing feature or a bug !