flagsmith-nodejs 3.1.0 → 3.1.1

package/build/sdk/index.d.ts

@@ -15,7 +15,6 @@ export declare class Flagsmith {
     customHeaders?: {
         [key: string]: any;
     };
-    requestTimeoutSeconds?: number;
     agent: RequestInit['agent'];
     requestTimeoutMs?: number;
     enableLocalEvaluation?: boolean;

package/build/sdk/index.js

@@ -89,6 +89,7 @@ Object.defineProperty(exports, "Flags", { enumerable: true, get: function () { r
 var polling_manager_2 = require("./polling_manager");
 Object.defineProperty(exports, "EnvironmentDataPollingManager", { enumerable: true, get: function () { return polling_manager_2.EnvironmentDataPollingManager; } });
 var DEFAULT_API_URL = 'https://edge.api.flagsmith.com/api/v1/';
+var DEFAULT_REQUEST_TIMEOUT_SECONDS = 10;
 var Flagsmith = /** @class */ (function () {
     /**
      * A Flagsmith client.
@@ -122,8 +123,8 @@ var Flagsmith = /** @class */ (function () {
         @param data.logger: an instance of the pino Logger class to use for logging
      */
     function Flagsmith(data) {
+        var _a;
         this.apiUrl = DEFAULT_API_URL;
-        this.requestTimeoutSeconds = 10;
         this.enableLocalEvaluation = false;
         this.environmentRefreshIntervalSeconds = 60;
         this.enableAnalytics = false;
@@ -131,8 +132,7 @@ var Flagsmith = /** @class */ (function () {
         this.environmentKey = data.environmentKey;
         this.apiUrl = data.apiUrl || this.apiUrl;
         this.customHeaders = data.customHeaders;
-        this.requestTimeoutSeconds = data.requestTimeoutSeconds;
-        this.requestTimeoutMs = data.requestTimeoutSeconds ? data.requestTimeoutSeconds * 1000 : undefined;
+        this.requestTimeoutMs = 1000 * ((_a = data.requestTimeoutSeconds) !== null && _a !== void 0 ? _a : DEFAULT_REQUEST_TIMEOUT_SECONDS);
         this.enableLocalEvaluation = data.enableLocalEvaluation;
         this.environmentRefreshIntervalSeconds =
             data.environmentRefreshIntervalSeconds || this.environmentRefreshIntervalSeconds;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "flagsmith-nodejs",
-  "version": "3.1.0",
+  "version": "3.1.1",
   "description": "Flagsmith lets you manage features flags and remote config across web, mobile and server side applications. Deliver true Continuous Integration. Get builds out faster. Control who has access to new features.",
   "main": "build/index.js",
   "repository": {

package/sdk/index.ts

@@ -24,13 +24,13 @@ export { EnvironmentDataPollingManager } from './polling_manager';
 export { FlagsmithCache, FlagsmithConfig } from './types';
 
 const DEFAULT_API_URL = 'https://edge.api.flagsmith.com/api/v1/';
+const DEFAULT_REQUEST_TIMEOUT_SECONDS = 10;
 
 
 export class Flagsmith {
     environmentKey?: string;
     apiUrl: string = DEFAULT_API_URL;
     customHeaders?: { [key: string]: any };
-    requestTimeoutSeconds?: number = 10;
     agent: RequestInit['agent'];
     requestTimeoutMs?: number;
     enableLocalEvaluation?: boolean = false;
@@ -87,8 +87,7 @@ export class Flagsmith {
         this.environmentKey = data.environmentKey;
         this.apiUrl = data.apiUrl || this.apiUrl;
         this.customHeaders = data.customHeaders;
-        this.requestTimeoutSeconds = data.requestTimeoutSeconds;
-        this.requestTimeoutMs = data.requestTimeoutSeconds ? data.requestTimeoutSeconds * 1000 : undefined;
+        this.requestTimeoutMs = 1000 * (data.requestTimeoutSeconds ?? DEFAULT_REQUEST_TIMEOUT_SECONDS);
         this.enableLocalEvaluation = data.enableLocalEvaluation;
         this.environmentRefreshIntervalSeconds =
             data.environmentRefreshIntervalSeconds || this.environmentRefreshIntervalSeconds;

package/tests/sdk/flagsmith.test.ts

@@ -183,7 +183,7 @@ test('default flag handler used when timeout occurs', async () => {
     const flg = new Flagsmith({
         environmentKey: 'key',
         defaultFlagHandler: defaultFlagHandler,
-        requestTimeoutSeconds: 0.1,
+        requestTimeoutSeconds: 0.001,
     });
 
     const flags = await flg.getEnvironmentFlags();
@@ -193,6 +193,15 @@ test('default flag handler used when timeout occurs', async () => {
     expect(flag.value).toBe(defaultFlag.value);
 })
 
+test('request timeout uses default if not provided', async () => {
+
+    const flg = new Flagsmith({
+        environmentKey: 'key',
+    });
+
+    expect(flg.requestTimeoutMs).toBe(10000);
+})
+
 test('test_throws_when_no_identity_flags_returned_due_to_error', async () => {
     // @ts-ignore
     fetch.mockReturnValue(Promise.resolve(new Response('bad data')));