~go-bot/goose/trunk

  • Committer: Tarmac
  • Author(s): John Arbash Meinel
  • Date: 2013-09-04 16:17:58 UTC
  • mfrom: (100.2.2 httpclient-ssl)
  • Revision ID: tarmac-20130904161758-ms9sbkj862taq2mt
[r=jameinel] http.Client: support InsecureSkipVerify

This adds a new API http.NewNonSSValidating which ends up passing
InsecureSkipVerify: true all the way down the http stack. Because we
were using the global stdlib net/http.DefaultClient to enable
connection sharing, etc, we continue to cache the http.Client object.
I only create the object once we actually need it, and we only use it
for requests that also want to be "insecure".

I'm open to suggestions if we want to name the API differently. We
could also do it as a flag to New() but that is an API break (meaning
juju-core stops building until we patch juju-core). So I felt this was
a reasonable method.

This is step 2 in my work to support ssl-hostname-verify:false for the
Openstack provider. (bug #1202163)

https://codereview.appspot.com/13520043/
Filename Latest Rev Last Changed Committer Comment Size
..
client.go 100.1.9 10 years ago John Arbash Meinel null-merge the httpsuite-ssl patch 8.6 KB Diff Download File
client_test.go 100.1.9 10 years ago John Arbash Meinel null-merge the httpsuite-ssl patch 5.7 KB Diff Download File