MOD:delete the buildPath code and use k8s native. And add a configGetter struct.

[calmkart]

this pr is about two things.

1. the code in forwardport to build the path is not required
   

   kubefwd/pkg/fwdport/fwdport.go

   Lines 62 to 67 in 8503882

   	u := url.URL{
   	Scheme: req.URL().Scheme,
   	Host: req.URL().Host,
   	Path: buildPath(req),
   	RawQuery: "timeout=32s",
   	}

   
   

   kubefwd/pkg/fwdport/fwdport.go

   Lines 174 to 178 in 8503882

   	func buildPath(req *restclient.Request) string {
   	splitted := strings.Split(req.URL().Path, "/namespaces")
   	path := splitted[0] + "/api/v1/namespaces" + splitted[1]
   	return path
   	}

   
   just use the k8s native way is well.

replace this

kubefwd/pkg/fwdport/fwdport.go

Line 55 in 8503882

restClient := pfo.ClientSet.RESTClient()

and change the way to get RESTClient with k8s native is better i think.

// GetRestClient return the RESTClient
func (c *ConfigGetter) GetRESTClient() (*restclient.RESTClient, error) {
	matchVersionKubeConfigFlags := cmdutil.NewMatchVersionFlags(c.ConfigFlag)
	f := cmdutil.NewFactory(matchVersionKubeConfigFlags)
	RESTClient, err := f.RESTClient()
	if err != nil {
		return nil, err
	}
	return RESTClient, nil
}

2. add a struct ConfigGetter in fwdcfg.go, maybe it will makes the code more concise.
3. I commented this line of code// restClient.Client.Timeout = 32 in fwdport.go#L55
   if we want to set the timeout, set here is well.

test in k8s v1.16.0

[calmkart]

@cjimti
PTAL,thanks.
if there is any question, pls contact me.
I am very happy to discuss it with you.