fix(supernode/httpclient): use HEAD request(not GET) to GetContentLength

liangran10000 · liangran10000 · commit 62a7ef268bec · 2020-09-29T15:48:30.000+08:00
diff --git a/supernode/httpclient/origin_http_client.go b/supernode/httpclient/origin_http_client.go
@@ -121,7 +121,7 @@ func (client *OriginClient) RegisterTLSConfig(rawURL string, insecure bool, caBl
 // GetContentLength sends a head request to get file length.
 func (client *OriginClient) GetContentLength(url string, headers map[string]string) (int64, int, error) {
 	// send request
-	resp, err := client.HTTPWithHeaders(http.MethodGet, url, headers, 4*time.Second)
+	resp, err := client.HTTPWithHeaders(http.MethodHead, url, headers, 4*time.Second)
 	if err != nil {
 		return 0, 0, err
 	}

Original file line number	Diff line number	Diff line change
`@@ -121,7 +121,7 @@ func (client *OriginClient) RegisterTLSConfig(rawURL string, insecure bool, caBl`
`121`	`121`	`// GetContentLength sends a head request to get file length.`
`122`	`122`	`func (client *OriginClient) GetContentLength(url string, headers map[string]string) (int64, int, error) {`
`123`	`123`	`// send request`
`124`		`- resp, err := client.HTTPWithHeaders(http.MethodGet, url, headers, 4*time.Second)`
	`124`	`+ resp, err := client.HTTPWithHeaders(http.MethodHead, url, headers, 4*time.Second)`
`125`	`125`	`if err != nil {`
`126`	`126`	`return 0, 0, err`
`127`	`127`	`}`