Merge pull request #34 from statsig-io/582sync

Patch sync interval reset

package.json

@@ -1,6 +1,6 @@
 {
   "name": "statsig-node",
-  "version": "5.8.2",
+  "version": "5.8.3",
   "description": "Statsig Node.js SDK for usage in multi-user server environments.",
   "main": "dist/index.js",
   "scripts": {
@@ -55,4 +55,4 @@
       "parser": "typescript"
     }
   }
-}
+}

src/SpecStore.ts

@@ -223,14 +223,12 @@ export default class SpecStore {
     let message = '';
     if (syncTimerInactive) {
       this.clearSyncTimer();
-      this._syncValues();
       message = message.concat(`Force reset sync timer. Last update time: ${
         this.syncTimerLastActiveTime
       }, now: ${Date.now()}`);
     }
     if (idListsSyncTimerInactive) {
       this.clearIdListsSyncTimer();
-      this._syncIdLists();
       message = message.concat(`Force reset id list sync timer. Last update time: ${
         this.idListsSyncTimerLastActiveTime
       }, now: ${Date.now()}`);
@@ -299,13 +297,15 @@ export default class SpecStore {
 
   private pollForUpdates() {
     if (this.syncTimer == null) {
+      this.syncTimerLastActiveTime = Date.now();
       this.syncTimer = poll(async () => {
         this.syncTimerLastActiveTime = Date.now();
         await this._syncValues();
       }, this.syncInterval);
     }
 
     if (this.idListsSyncTimer == null) {
+      this.idListsSyncTimerLastActiveTime = Date.now();
       this.idListsSyncTimer = poll(async () => {
         this.idListsSyncTimerLastActiveTime = Date.now();
         await this._syncIdLists();

src/__tests__/ResetSync.test.ts

@@ -88,6 +88,18 @@ describe('Verify sync intervals reset', () => {
       { userID: '123', email: '[email protected]' },
       'nfl_gate',
     );
+    gate = await statsig.checkGate(
+      { userID: '123', email: '[email protected]' },
+      'nfl_gate',
+    );
+    gate = await statsig.checkGate(
+      { userID: '123', email: '[email protected]' },
+      'nfl_gate',
+    );
+    gate = await statsig.checkGate(
+      { userID: '123', email: '[email protected]' },
+      'nfl_gate',
+    );
     expect(gate).toBe(true);
     expect(spy).toHaveBeenCalledTimes(1);
   });

src/utils/StatsigFetcher.ts

@@ -56,7 +56,7 @@ export default class StatsigFetcher {
     if (counter != null && counter >= 1000) {
       return Promise.reject(
         new StatsigTooManyRequestsError(
-          'Request failed because you are making the same request too frequently.',
+          `Request to ${url} failed because you are making the same request too frequently (${counter}).`,
         ),
       );
     }