split util to specific packages (#2)

pull/1670/head
Anton Galitsyn 9 years ago committed by GitHub
parent de555b4eb3
commit a181943fb8

@ -23,7 +23,7 @@ import (
"k8s.io/helm/cmd/helm/downloader" "k8s.io/helm/cmd/helm/downloader"
"k8s.io/helm/cmd/helm/helmpath" "k8s.io/helm/cmd/helm/helmpath"
"k8s.io/helm/pkg/util" "k8s.io/helm/pkg/httputil"
) )
const dependencyBuildDesc = ` const dependencyBuildDesc = `
@ -84,7 +84,7 @@ func (d *dependencyBuildCmd) run() error {
var client *http.Client var client *http.Client
var err error var err error
if d.certFile != "" && d.keyFile != "" && d.caFile != "" { if d.certFile != "" && d.keyFile != "" && d.caFile != "" {
client, err = util.NewHTTPClientTLS(d.certFile, d.keyFile, d.caFile) client, err = httputil.NewHTTPClientTLS(d.certFile, d.keyFile, d.caFile)
if err != nil { if err != nil {
return err return err
} }

@ -23,7 +23,7 @@ import (
"github.com/spf13/cobra" "github.com/spf13/cobra"
"k8s.io/helm/cmd/helm/downloader" "k8s.io/helm/cmd/helm/downloader"
"k8s.io/helm/cmd/helm/helmpath" "k8s.io/helm/cmd/helm/helmpath"
"k8s.io/helm/pkg/util" "k8s.io/helm/pkg/httputil"
) )
const dependencyUpDesc = ` const dependencyUpDesc = `
@ -94,7 +94,7 @@ func (d *dependencyUpdateCmd) run() error {
var client *http.Client var client *http.Client
var err error var err error
if d.certFile != "" && d.keyFile != "" && d.caFile != "" { if d.certFile != "" && d.keyFile != "" && d.caFile != "" {
client, err = util.NewHTTPClientTLS(d.certFile, d.keyFile, d.caFile) client, err = httputil.NewHTTPClientTLS(d.certFile, d.keyFile, d.caFile)
if err != nil { if err != nil {
return err return err
} }

@ -28,7 +28,7 @@ import (
"k8s.io/helm/cmd/helm/downloader" "k8s.io/helm/cmd/helm/downloader"
"k8s.io/helm/cmd/helm/helmpath" "k8s.io/helm/cmd/helm/helmpath"
"k8s.io/helm/pkg/chartutil" "k8s.io/helm/pkg/chartutil"
"k8s.io/helm/pkg/util" "k8s.io/helm/pkg/httputil"
) )
const fetchDesc = ` const fetchDesc = `
@ -104,7 +104,7 @@ func (f *fetchCmd) run() error {
var client *http.Client var client *http.Client
var err error var err error
if f.certFile != "" && f.keyFile != "" && f.caFile != "" { if f.certFile != "" && f.keyFile != "" && f.caFile != "" {
client, err = util.NewHTTPClientTLS(f.certFile, f.keyFile, f.caFile) client, err = httputil.NewHTTPClientTLS(f.certFile, f.keyFile, f.caFile)
if err != nil { if err != nil {
return err return err
} }

@ -26,8 +26,8 @@ import (
"github.com/spf13/cobra" "github.com/spf13/cobra"
"k8s.io/helm/cmd/helm/helmpath" "k8s.io/helm/cmd/helm/helmpath"
"k8s.io/helm/pkg/httputil"
"k8s.io/helm/pkg/repo" "k8s.io/helm/pkg/repo"
"k8s.io/helm/pkg/util"
) )
type repoAddCmd struct { type repoAddCmd struct {
@ -77,7 +77,7 @@ func (a *repoAddCmd) run() error {
var client *http.Client var client *http.Client
var err error var err error
if a.certFile != "" && a.keyFile != "" && a.caFile != "" { if a.certFile != "" && a.keyFile != "" && a.caFile != "" {
client, err = util.NewHTTPClientTLS(a.certFile, a.keyFile, a.caFile) client, err = httputil.NewHTTPClientTLS(a.certFile, a.keyFile, a.caFile)
if err != nil { if err != nil {
return err return err
} }

@ -26,8 +26,8 @@ import (
"github.com/spf13/cobra" "github.com/spf13/cobra"
"k8s.io/helm/cmd/helm/helmpath" "k8s.io/helm/cmd/helm/helmpath"
"k8s.io/helm/pkg/httputil"
"k8s.io/helm/pkg/repo" "k8s.io/helm/pkg/repo"
"k8s.io/helm/pkg/util"
) )
const updateDesc = ` const updateDesc = `
@ -77,7 +77,7 @@ func (u *repoUpdateCmd) run() error {
var client *http.Client var client *http.Client
var err error var err error
if u.certFile != "" && u.keyFile != "" && u.caFile != "" { if u.certFile != "" && u.keyFile != "" && u.caFile != "" {
client, err = util.NewHTTPClientTLS(u.certFile, u.keyFile, u.caFile) client, err = httputil.NewHTTPClientTLS(u.certFile, u.keyFile, u.caFile)
if err != nil { if err != nil {
return err return err
} }

@ -14,16 +14,18 @@ See the License for the specific language governing permissions and
limitations under the License. limitations under the License.
*/ */
package util package httputil
import ( import (
"fmt" "fmt"
"net/http" "net/http"
"k8s.io/helm/pkg/tlsutil"
) )
// NewHTTPClientTLS constructs http.Client with configured TLS for http.Transport // NewHTTPClientTLS constructs http.Client with configured TLS for http.Transport
func NewHTTPClientTLS(certFile, keyFile, caFile string) (*http.Client, error) { func NewHTTPClientTLS(certFile, keyFile, caFile string) (*http.Client, error) {
tlsConf, err := NewClientTLS(certFile, keyFile, caFile) tlsConf, err := tlsutil.NewClientTLS(certFile, keyFile, caFile)
if err != nil { if err != nil {
return nil, fmt.Errorf("can't create TLS config for client: %s", err.Error()) return nil, fmt.Errorf("can't create TLS config for client: %s", err.Error())
} }

@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
limitations under the License. limitations under the License.
*/ */
package util package tlsutil
import ( import (
"crypto/tls" "crypto/tls"
Loading…
Cancel
Save