Created by: keegancsmith
This commit audited all our direct uses of http.Client and replaced them with httpcli.DefaultExternalClient if they communicated with external services (ie outside of cluster / docker container services). Additionally we also audited uses of ctxhttp since passing in nil uses http.DefaultClient.
This can still miss some uses. In particular it will miss uses of http.DefaultClient inside of dependencies. There are some likely sources of this already, but this will get us most of the way there. We can follow-up with these issues as they come up / with testing in the cycle.
Potential http.DefaultClient:
Note: We introduce httpcli.Client to simplify the call sites which currently use ctxhttp or http.DefaultClient.
Part of #71 (closed) Part of #4652 (closed)