Great article, thanks!

There was one part that seemed very un-Go like though, and that was checking if the http.Get() response body is nil even on error.

The http documentation states that:

On error, any Response can be ignored. A non-nil Response with a non-nil error only occurs when CheckRedirect fails, and even then the returned Response.Body is already closed.

Clearly there’s no need to do this and we shouldn’t advice users to do so either.

Regards,
Mathias