Severe performance degredation using TraceLoggingTransport

[bkuschel]

When uploading or downloading large streams of data, the tracing facility imposes a large tax. Perhaps the default should allow it to be disabled.

Work around is to define a cluster HTTPClient in the config without the trace wrapper.

func NewHTTPClient(config *ibmcloud.Config) *http.Client {
	return &http.Client{
		Transport: makeTransport(config),
		Timeout:   config.HTTPTimeout,
	}
}

func makeTransport(config *ibmcloud.Config) http.RoundTripper {
	return &http.Transport{
		Proxy: http.ProxyFromEnvironment,
		Dial: (&net.Dialer{
			Timeout:   50 * time.Second,
			KeepAlive: 30 * time.Second,
		}).Dial,
		TLSHandshakeTimeout: 20 * time.Second,
		DisableCompression:  true,
		TLSClientConfig: &tls.Config{
			InsecureSkipVerify: config.SSLDisable,
		},
	}
}
...
c.HTTPConfig = NewHTTPClient(c)