Merge pull request kubernetes#88064 from wongma7/webhook-header

k8s-ci-robot · web-flow · commit 1aa21639cf16 · 2020-02-12T19:54:35.000-08:00
Initialize http Request Header before RoundTrip to avoid panic
diff --git a/test/e2e/framework/util.go b/test/e2e/framework/util.go
@@ -1536,7 +1536,7 @@ func headersForConfig(c *restclient.Config, url *url.URL) (http.Header, error) {
 	if err != nil {
 		return nil, err
 	}
-	if _, err := rt.RoundTrip(&http.Request{URL: url}); err != nil {
+	if _, err := rt.RoundTrip(&http.Request{Header: make(http.Header), URL: url}); err != nil {
 		return nil, err
 	}
 	return extract.Header, nil

Original file line number	Diff line number	Diff line change
`@@ -1536,7 +1536,7 @@ func headersForConfig(c restclient.Config, url url.URL) (http.Header, error) {`
`1536`	`1536`	`if err != nil {`
`1537`	`1537`	`return nil, err`
`1538`	`1538`	`}`
`1539`		`- if _, err := rt.RoundTrip(&http.Request{URL: url}); err != nil {`
	`1539`	`+ if _, err := rt.RoundTrip(&http.Request{Header: make(http.Header), URL: url}); err != nil {`
`1540`	`1540`	`return nil, err`
`1541`	`1541`	`}`
`1542`	`1542`	`return extract.Header, nil`