Skip to content

Commit deaf7e3

Browse files
cfergeaupraveenkumar
authored andcommitted
move HTTPTransport and proxy related code to its own package httpproxy
1 parent 8e08162 commit deaf7e3

File tree

16 files changed

+38
-35
lines changed

16 files changed

+38
-35
lines changed

cmd/crc/cmd/root.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ import (
1515
crcErr "github.com/crc-org/crc/pkg/crc/errors"
1616
"github.com/crc-org/crc/pkg/crc/logging"
1717
"github.com/crc-org/crc/pkg/crc/machine"
18-
"github.com/crc-org/crc/pkg/crc/network"
18+
"github.com/crc-org/crc/pkg/crc/network/httpproxy"
1919
"github.com/crc-org/crc/pkg/crc/preflight"
2020
"github.com/crc-org/crc/pkg/crc/segment"
2121
"github.com/crc-org/crc/pkg/crc/telemetry"
@@ -60,7 +60,7 @@ func init() {
6060
}
6161

6262
// Initiate segment client
63-
if segmentClient, err = segment.NewClient(config, network.HTTPTransport()); err != nil {
63+
if segmentClient, err = segment.NewClient(config, httpproxy.HTTPTransport()); err != nil {
6464
logging.Fatal(err.Error())
6565
}
6666

@@ -132,7 +132,7 @@ func setProxyDefaults() error {
132132
noProxy := config.Get(crcConfig.NoProxy).AsString()
133133
proxyCAFile := config.Get(crcConfig.ProxyCAFile).AsString()
134134

135-
proxyConfig, err := network.NewProxyDefaults(httpProxy, httpsProxy, noProxy, proxyCAFile)
135+
proxyConfig, err := httpproxy.NewProxyDefaults(httpProxy, httpsProxy, noProxy, proxyCAFile)
136136
if err != nil {
137137
return err
138138
}

cmd/crc/cmd/start.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ import (
1919
"github.com/crc-org/crc/pkg/crc/logging"
2020
"github.com/crc-org/crc/pkg/crc/machine/types"
2121
"github.com/crc-org/crc/pkg/crc/network"
22+
"github.com/crc-org/crc/pkg/crc/network/httpproxy"
2223
"github.com/crc-org/crc/pkg/crc/preflight"
2324
"github.com/crc-org/crc/pkg/crc/preset"
2425
"github.com/crc-org/crc/pkg/crc/validation"
@@ -211,7 +212,7 @@ func checkIfNewVersionAvailable(noUpdateCheck bool) error {
211212
}
212213

213214
func newVersionAvailable() (bool, string, string, error) {
214-
release, err := crcversion.GetCRCLatestVersionFromMirror(network.HTTPTransport())
215+
release, err := crcversion.GetCRCLatestVersionFromMirror(httpproxy.HTTPTransport())
215216
if err != nil {
216217
return false, "", "", err
217218
}

pkg/crc/cluster/cluster.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ import (
1616
"github.com/crc-org/crc/pkg/crc/constants"
1717
"github.com/crc-org/crc/pkg/crc/errors"
1818
"github.com/crc-org/crc/pkg/crc/logging"
19-
"github.com/crc-org/crc/pkg/crc/network"
19+
"github.com/crc-org/crc/pkg/crc/network/httpproxy"
2020
"github.com/crc-org/crc/pkg/crc/oc"
2121
"github.com/crc-org/crc/pkg/crc/ssh"
2222
crctls "github.com/crc-org/crc/pkg/crc/tls"
@@ -322,7 +322,7 @@ func EnsureClusterIDIsNotEmpty(ctx context.Context, ocConfig oc.Config) error {
322322
return nil
323323
}
324324

325-
func AddProxyConfigToCluster(ctx context.Context, sshRunner *ssh.Runner, ocConfig oc.Config, proxy *network.ProxyConfig) error {
325+
func AddProxyConfigToCluster(ctx context.Context, sshRunner *ssh.Runner, ocConfig oc.Config, proxy *httpproxy.ProxyConfig) error {
326326
type trustedCA struct {
327327
Name string `json:"name"`
328328
}
@@ -372,7 +372,7 @@ func AddProxyConfigToCluster(ctx context.Context, sshRunner *ssh.Runner, ocConfi
372372
return nil
373373
}
374374

375-
func addProxyCACertToCluster(sshRunner *ssh.Runner, ocConfig oc.Config, proxy *network.ProxyConfig, trustedCAName string) error {
375+
func addProxyCACertToCluster(sshRunner *ssh.Runner, ocConfig oc.Config, proxy *httpproxy.ProxyConfig, trustedCAName string) error {
376376
proxyConfigMapFileName := fmt.Sprintf("/tmp/%s.json", trustedCAName)
377377
proxyCABundleTemplate := `{
378378
"apiVersion": "v1",
@@ -476,7 +476,7 @@ func DeleteOpenshiftAPIServerPods(ctx context.Context, ocConfig oc.Config) error
476476
return errors.Retry(ctx, 60*time.Second, deleteOpenshiftAPIServerPods, time.Second)
477477
}
478478

479-
func CheckProxySettingsForOperator(ocConfig oc.Config, proxy *network.ProxyConfig, deployment, namespace string) (bool, error) {
479+
func CheckProxySettingsForOperator(ocConfig oc.Config, proxy *httpproxy.ProxyConfig, deployment, namespace string) (bool, error) {
480480
if !proxy.IsEnabled() {
481481
logging.Debugf("No proxy in use")
482482
return true, nil

pkg/crc/cluster/status.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,11 @@ import (
66
"time"
77

88
"github.com/crc-org/crc/pkg/crc/logging"
9-
"github.com/crc-org/crc/pkg/crc/network"
9+
"github.com/crc-org/crc/pkg/crc/network/httpproxy"
1010
)
1111

1212
// WaitForClusterStable checks that the cluster is running a number of consecutive times
13-
func WaitForClusterStable(ctx context.Context, ip string, kubeconfigFilePath string, proxy *network.ProxyConfig) error {
13+
func WaitForClusterStable(ctx context.Context, ip string, kubeconfigFilePath string, proxy *httpproxy.ProxyConfig) error {
1414
if ctx.Err() != nil {
1515
return ctx.Err()
1616
}

pkg/crc/config/validations.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import (
55
"strings"
66

77
"github.com/crc-org/crc/pkg/crc/constants"
8-
"github.com/crc-org/crc/pkg/crc/network"
8+
"github.com/crc-org/crc/pkg/crc/network/httpproxy"
99
crcpreset "github.com/crc-org/crc/pkg/crc/preset"
1010
"github.com/crc-org/crc/pkg/crc/validation"
1111
"github.com/spf13/cast"
@@ -92,15 +92,15 @@ func validatePath(value interface{}) (bool, string) {
9292

9393
// validateHTTPProxy checks if given URI is valid for a HTTP proxy
9494
func validateHTTPProxy(value interface{}) (bool, string) {
95-
if err := network.ValidateProxyURL(cast.ToString(value), false); err != nil {
95+
if err := httpproxy.ValidateProxyURL(cast.ToString(value), false); err != nil {
9696
return false, err.Error()
9797
}
9898
return true, ""
9999
}
100100

101101
// validateHTTPSProxy checks if given URI is valid for a HTTPS proxy
102102
func validateHTTPSProxy(value interface{}) (bool, string) {
103-
if err := network.ValidateProxyURL(cast.ToString(value), true); err != nil {
103+
if err := httpproxy.ValidateProxyURL(cast.ToString(value), true); err != nil {
104104
return false, err.Error()
105105
}
106106
return true, ""

pkg/crc/machine/bundle/metadata.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ import (
1818
"github.com/crc-org/crc/pkg/crc/gpg"
1919
"github.com/crc-org/crc/pkg/crc/image"
2020
"github.com/crc-org/crc/pkg/crc/logging"
21-
"github.com/crc-org/crc/pkg/crc/network"
21+
"github.com/crc-org/crc/pkg/crc/network/httpproxy"
2222
crcPreset "github.com/crc-org/crc/pkg/crc/preset"
2323
"github.com/crc-org/crc/pkg/download"
2424
)
@@ -301,7 +301,7 @@ func getBundleDownloadInfo(preset crcPreset.Preset) (*download.RemoteFile, error
301301
func getDefaultBundleVerifiedHash(preset crcPreset.Preset) (string, error) {
302302
client := &http.Client{
303303
Timeout: 5 * time.Second,
304-
Transport: network.HTTPTransport(),
304+
Transport: httpproxy.HTTPTransport(),
305305
}
306306
res, err := client.Get(constants.GetDefaultBundleSignedHashURL(preset))
307307
if err != nil {

pkg/crc/machine/fakemachine/client.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import (
66

77
"github.com/crc-org/crc/pkg/crc/machine/state"
88
"github.com/crc-org/crc/pkg/crc/machine/types"
9-
"github.com/crc-org/crc/pkg/crc/network"
9+
"github.com/crc-org/crc/pkg/crc/network/httpproxy"
1010
"github.com/crc-org/crc/pkg/crc/preset"
1111
)
1212

@@ -55,7 +55,7 @@ func (c *Client) GetConsoleURL() (*types.ConsoleResult, error) {
5555
}, nil
5656
}
5757

58-
func (c *Client) GetProxyConfig(machineName string) (*network.ProxyConfig, error) {
58+
func (c *Client) GetProxyConfig(machineName string) (*httpproxy.ProxyConfig, error) {
5959
return nil, errors.New("not implemented")
6060
}
6161

pkg/crc/machine/machine.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import (
88
"github.com/crc-org/crc/pkg/crc/constants"
99
"github.com/crc-org/crc/pkg/crc/machine/bundle"
1010
"github.com/crc-org/crc/pkg/crc/machine/types"
11-
"github.com/crc-org/crc/pkg/crc/network"
11+
"github.com/crc-org/crc/pkg/crc/network/httpproxy"
1212
"github.com/crc-org/crc/pkg/libmachine"
1313
"github.com/crc-org/machine/libmachine/drivers"
1414
)
@@ -17,7 +17,7 @@ func getClusterConfig(bundleInfo *bundle.CrcBundleInfo) (*types.ClusterConfig, e
1717
if !bundleInfo.IsOpenShift() {
1818
return &types.ClusterConfig{
1919
ClusterType: bundleInfo.GetBundleType(),
20-
ProxyConfig: &network.ProxyConfig{},
20+
ProxyConfig: &httpproxy.ProxyConfig{},
2121
}, nil
2222
}
2323

@@ -65,8 +65,8 @@ func createLibMachineClient() (libmachine.API, func()) {
6565
}
6666
}
6767

68-
func getProxyConfig(bundleInfo *bundle.CrcBundleInfo) (*network.ProxyConfig, error) {
69-
proxy, err := network.NewProxyConfig()
68+
func getProxyConfig(bundleInfo *bundle.CrcBundleInfo) (*httpproxy.ProxyConfig, error) {
69+
proxy, err := httpproxy.NewProxyConfig()
7070
if err != nil {
7171
return nil, err
7272
}

pkg/crc/machine/start.go

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ import (
2121
"github.com/crc-org/crc/pkg/crc/machine/state"
2222
"github.com/crc-org/crc/pkg/crc/machine/types"
2323
"github.com/crc-org/crc/pkg/crc/network"
24+
"github.com/crc-org/crc/pkg/crc/network/httpproxy"
2425
"github.com/crc-org/crc/pkg/crc/oc"
2526
"github.com/crc-org/crc/pkg/crc/podman"
2627
crcPreset "github.com/crc-org/crc/pkg/crc/preset"
@@ -761,7 +762,7 @@ func copyKubeconfigFileWithUpdatedUserClientCertAndKey(selfSignedCAKey *rsa.Priv
761762
return updateClientCrtAndKeyToKubeconfig(clientKey, clientCert, srcKubeConfigPath, dstKubeConfigPath)
762763
}
763764

764-
func configurePodmanProxy(ctx context.Context, sshRunner *crcssh.Runner, proxy *network.ProxyConfig) (err error) {
765+
func configurePodmanProxy(ctx context.Context, sshRunner *crcssh.Runner, proxy *httpproxy.ProxyConfig) (err error) {
765766
if !proxy.IsEnabled() {
766767
return nil
767768
}
@@ -813,15 +814,15 @@ func configurePodmanProxy(ctx context.Context, sshRunner *crcssh.Runner, proxy *
813814

814815
}
815816

816-
func ensureProxyIsConfiguredInOpenShift(ctx context.Context, ocConfig oc.Config, sshRunner *crcssh.Runner, proxy *network.ProxyConfig) (err error) {
817+
func ensureProxyIsConfiguredInOpenShift(ctx context.Context, ocConfig oc.Config, sshRunner *crcssh.Runner, proxy *httpproxy.ProxyConfig) (err error) {
817818
if !proxy.IsEnabled() {
818819
return nil
819820
}
820821
logging.Info("Adding proxy configuration to the cluster...")
821822
return cluster.AddProxyConfigToCluster(ctx, sshRunner, ocConfig, proxy)
822823
}
823824

824-
func waitForProxyPropagation(ctx context.Context, ocConfig oc.Config, proxyConfig *network.ProxyConfig) {
825+
func waitForProxyPropagation(ctx context.Context, ocConfig oc.Config, proxyConfig *httpproxy.ProxyConfig) {
825826
if !proxyConfig.IsEnabled() {
826827
return
827828
}

pkg/crc/machine/types/types.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ package types
33
import (
44
"github.com/crc-org/crc/pkg/crc/cluster"
55
"github.com/crc-org/crc/pkg/crc/machine/state"
6-
"github.com/crc-org/crc/pkg/crc/network"
6+
"github.com/crc-org/crc/pkg/crc/network/httpproxy"
77
"github.com/crc-org/crc/pkg/crc/preset"
88
crcpreset "github.com/crc-org/crc/pkg/crc/preset"
99
)
@@ -46,7 +46,7 @@ type ClusterConfig struct {
4646
KubeAdminPass string
4747
ClusterAPI string
4848
WebConsoleURL string
49-
ProxyConfig *network.ProxyConfig
49+
ProxyConfig *httpproxy.ProxyConfig
5050
}
5151

5252
type StartResult struct {

0 commit comments

Comments
 (0)