https://codereview.appspot.com/56560043/diff/20001/provider/common/state.go#newcode71
provider/common/state.go:71: func LoadStateFromURL(url string,
disableSSLHostnameVerification bool) (*BootstrapState, error) {
On 2014/01/29 08:12:59, axw wrote:
> This is fine for now, but I wonder if we shouldn't have a method of
obtaining an
> http.Client for an environment. We now have HTTP proxy settings and
SSL hostname
> verification as options. We could centralise all that and pass an
http.Client in
> here.
Yes, on trunk there should be some refactoring like that. There are a
couple of places that want a http.Client obeying thig setting and we may
want more.
Please take a look.
https:/ /codereview. appspot. com/56560043/ diff/20001/ environs/ httpstorage/ storage. go httpstorage/ storage. go (right):
File environs/
https:/ /codereview. appspot. com/56560043/ diff/20001/ environs/ httpstorage/ storage. go#newcode26 httpstorage/ storage. go:26: type PromotableStorage interface { httputil. ReverseProxy.
environs/
On 2014/01/29 08:12:59, axw wrote:
> I'm not keen on this. I think a better way to test, rather than
further exposing
> the guts of this package, would be to use a
net/http/
Okay, I'll revert most of this and write something independant.
https:/ /codereview. appspot. com/56560043/ diff/20001/ provider/ common/ state.go common/ state.go (right):
File provider/
https:/ /codereview. appspot. com/56560043/ diff/20001/ provider/ common/ state.go# newcode71 common/ state.go: 71: func LoadStateFromUR L(url string, ameVerification bool) (*BootstrapState, error) {
provider/
disableSSLHostn
On 2014/01/29 08:12:59, axw wrote:
> This is fine for now, but I wonder if we shouldn't have a method of
obtaining an
> http.Client for an environment. We now have HTTP proxy settings and
SSL hostname
> verification as options. We could centralise all that and pass an
http.Client in
> here.
Yes, on trunk there should be some refactoring like that. There are a
couple of places that want a http.Client obeying thig setting and we may
want more.
https:/ /codereview. appspot. com/56560043/