@naturalcycles/js-lib 14.133.1 → 14.134.0

package/dist/http/fetcher.d.ts

@@ -42,6 +42,8 @@ export declare class Fetcher {
     /**
      * Default is yes,
      * unless there's reason not to (e.g method is POST).
+     *
+     * statusCode of 0 (or absense of it) will BE retried.
      */
     private shouldRetry;
     private getStatusFamily;

package/dist/http/fetcher.js

@@ -12,7 +12,7 @@ const time_util_1 = require("../time/time.util");
 const http_model_1 = require("./http.model");
 const defRetryOptions = {
     count: 2,
-    timeout: 500,
+    timeout: 1000,
     timeoutMax: 30000,
     timeoutMultiplier: 2,
 };
@@ -253,11 +253,14 @@ class Fetcher {
             return;
         retryStatus.retryAttempt++;
         retryStatus.retryTimeout = (0, number_util_1._clamp)(retryStatus.retryTimeout * timeoutMultiplier, 0, timeoutMax);
-        await (0, pDelay_1.pDelay)(retryStatus.retryTimeout);
+        const noise = Math.random() * 500;
+        await (0, pDelay_1.pDelay)(retryStatus.retryTimeout + noise);
     }
     /**
      * Default is yes,
      * unless there's reason not to (e.g method is POST).
+     *
+     * statusCode of 0 (or absense of it) will BE retried.
      */
     shouldRetry(res) {
         const { retryPost, retry4xx, retry5xx } = res.req;

package/dist/string/stringifyAny.js

@@ -81,6 +81,8 @@ function _stringifyAny(obj, opt = {}) {
         }
         if ((0, error_util_1._isErrorObject)(obj)) {
             if ((0, error_util_1._isHttpErrorObject)(obj)) {
+                // Only include (statusCode) if it's non-zero
+                // No: print (0), as it removes ambiguity
                 // `replace` here works ONCE, exactly as we need it
                 s = s.replace('HttpError', `HttpError(${obj.data.httpStatusCode})`);
             }

package/dist-esm/http/fetcher.js

@@ -10,7 +10,7 @@ import { _since } from '../time/time.util';
 import { HTTP_METHODS } from './http.model';
 const defRetryOptions = {
     count: 2,
-    timeout: 500,
+    timeout: 1000,
     timeoutMax: 30000,
     timeoutMultiplier: 2,
 };
@@ -291,11 +291,14 @@ export class Fetcher {
             return;
         retryStatus.retryAttempt++;
         retryStatus.retryTimeout = _clamp(retryStatus.retryTimeout * timeoutMultiplier, 0, timeoutMax);
-        await pDelay(retryStatus.retryTimeout);
+        const noise = Math.random() * 500;
+        await pDelay(retryStatus.retryTimeout + noise);
     }
     /**
      * Default is yes,
      * unless there's reason not to (e.g method is POST).
+     *
+     * statusCode of 0 (or absense of it) will BE retried.
      */
     shouldRetry(res) {
         var _a;

package/dist-esm/string/stringifyAny.js

@@ -77,6 +77,8 @@ export function _stringifyAny(obj, opt = {}) {
         }
         if (_isErrorObject(obj)) {
             if (_isHttpErrorObject(obj)) {
+                // Only include (statusCode) if it's non-zero
+                // No: print (0), as it removes ambiguity
                 // `replace` here works ONCE, exactly as we need it
                 s = s.replace('HttpError', `HttpError(${obj.data.httpStatusCode})`);
             }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@naturalcycles/js-lib",
-  "version": "14.133.1",
+  "version": "14.134.0",
   "scripts": {
     "prepare": "husky install",
     "build-prod": "build-prod-esm-cjs",

package/src/http/fetcher.ts

@@ -30,7 +30,7 @@ import type { HttpStatusFamily } from './http.model'
 
 const defRetryOptions: FetcherRetryOptions = {
   count: 2,
-  timeout: 500,
+  timeout: 1000,
   timeoutMax: 30_000,
   timeoutMultiplier: 2,
 }
@@ -336,12 +336,15 @@ export class Fetcher {
     retryStatus.retryAttempt++
     retryStatus.retryTimeout = _clamp(retryStatus.retryTimeout * timeoutMultiplier, 0, timeoutMax)
 
-    await pDelay(retryStatus.retryTimeout)
+    const noise = Math.random() * 500
+    await pDelay(retryStatus.retryTimeout + noise)
   }
 
   /**
    * Default is yes,
    * unless there's reason not to (e.g method is POST).
+   *
+   * statusCode of 0 (or absense of it) will BE retried.
    */
   private shouldRetry(res: FetcherResponse): boolean {
     const { retryPost, retry4xx, retry5xx } = res.req

package/src/string/stringifyAny.ts

@@ -125,6 +125,8 @@ export function _stringifyAny(obj: any, opt: StringifyAnyOptions = {}): string {
 
     if (_isErrorObject(obj)) {
       if (_isHttpErrorObject(obj)) {
+        // Only include (statusCode) if it's non-zero
+        // No: print (0), as it removes ambiguity
         // `replace` here works ONCE, exactly as we need it
         s = s.replace('HttpError', `HttpError(${obj.data.httpStatusCode})`)
       }