Merge pull request #175 from eunomie/fix-login

glours · web-flow · commit 9ef288b535bb · 2021-11-17T21:46:09.000+01:00
fix: remove NopCloser to get the right content-length set
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`	`}`