Merge pull request #25 from configcat/fix-infinite-callback-loop

Fix infinite callback loop when autpoll fetch fails with empty cache

DEPLOY.md

@@ -4,7 +4,7 @@
 
 1. Run `./deploy.sh`
 
-2. Update `common-js` in `js-sdk` and `node-sdk` and re-deploy both packages.
+2. Update `common-js` in `js-sdk` and `node-sdk` and `js-ssr-sdk` and re-deploy both packages.
 
 or
 

src/AutoPollConfigService.ts

@@ -43,8 +43,10 @@ export class AutoPollConfigService extends ConfigServiceBase implements IConfigS
 
             const newConfig = await this.refreshLogicBaseAsync(cachedConfig, forceUpdateCache)
 
-            if (!cachedConfig || !ProjectConfig.equals(cachedConfig, newConfig)) {
-
+            let weDontHaveCachedYetButHaveNew = !cachedConfig && newConfig;
+            let weHaveBothButTheyDiffers = cachedConfig && newConfig && !ProjectConfig.equals(cachedConfig, newConfig);
+
+            if (weDontHaveCachedYetButHaveNew || weHaveBothButTheyDiffers) {                
                 this.configChanged();
             }