fix: remove NopCloser to get the right content-length set

eunomie · eunomie · commit 1b21586da4d4 · 2021-11-17T18:35:52.000+01:00
Signed-off-by: Yves Brissaud &lt;yves.brissaud@docker.com&gt;
diff --git a/internal/hub/hub.go b/internal/hub/hub.go
@@ -47,7 +47,7 @@ func (h *Client) Login(hubAuthConfig types.AuthConfig) (string, error) {
 	body := bytes.NewBuffer(data)
 
 	// Login on the Docker Hub
-	req, err := http.NewRequest("POST", h.Domain+LoginURL, ioutil.NopCloser(body))
+	req, err := http.NewRequest("POST", h.Domain+LoginURL, body)
 	if err != nil {
 		return "", err
 	}

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ func (h *Client) Login(hubAuthConfig types.AuthConfig) (string, error) {`
`47`	`47`	`body := bytes.NewBuffer(data)`
`48`	`48`
`49`	`49`	`// Login on the Docker Hub`
`50`		`- req, err := http.NewRequest("POST", h.Domain+LoginURL, ioutil.NopCloser(body))`
	`50`	`+ req, err := http.NewRequest("POST", h.Domain+LoginURL, body)`
`51`	`51`	`if err != nil {`
`52`	`52`	`return "", err`
`53`	`53`	`}`