fix: avoid setting body for GET requests

When making a GET request to certain uplinks, such as https://registry.npmmirror.com, setting the body field can result in a 413 error. Previously, the code was setting the body field for all requests, including GET requests.

This commit fixes the issue by checking the request method and avoiding setting the body field for GET requests. This ensures that GET requests are not affected by the issue and can be made without error.

Fixes verdaccio#3601

src/lib/up-storage.ts

@@ -219,7 +219,6 @@ class ProxyStorage implements IProxy {
       url: uri,
       method: method,
       headers: headers,
-      body: json,
       proxy: this.proxy,
       encoding: null,
       gzip: true,
@@ -228,6 +227,11 @@ class ProxyStorage implements IProxy {
       agentOptions: this.agent_options,
     };
 
+    // GET requests should not have a body, otherwise the request might fail(return 413 status code)
+    if (method.toUpperCase() !== 'GET') {
+      requestOptions.body = json
+    }
+
     if (this.ca) {
       requestOptions = Object.assign({}, requestOptions, {
         ca: this.ca,