@splitsoftware/splitio-commons 2.7.9-rc.3 → 2.8.1-rc.0

package/esm/evaluator/fallbackTreatmentsCalculator/fallbackSanitizer/index.js

@@ -4,45 +4,51 @@ var FallbackDiscardReason;
     FallbackDiscardReason["FlagName"] = "Invalid flag name (max 100 chars, no spaces)";
     FallbackDiscardReason["Treatment"] = "Invalid treatment (max 100 chars and must match pattern)";
 })(FallbackDiscardReason || (FallbackDiscardReason = {}));
-var FallbacksSanitizer = /** @class */ (function () {
-    function FallbacksSanitizer() {
+var TREATMENT_PATTERN = /^[0-9]+[.a-zA-Z0-9_-]*$|^[a-zA-Z]+[a-zA-Z0-9_-]*$/;
+export function isValidFlagName(name) {
+    return name.length <= 100 && !name.includes(' ');
+}
+export function isValidTreatment(t) {
+    var treatment = isObject(t) ? t.treatment : t;
+    if (!isString(treatment) || treatment.length > 100) {
+        return false;
     }
-    FallbacksSanitizer.isValidFlagName = function (name) {
-        return name.length <= 100 && !name.includes(' ');
-    };
-    FallbacksSanitizer.isValidTreatment = function (t) {
-        var treatment = isObject(t) ? t.treatment : t;
-        if (!isString(treatment) || treatment.length > 100) {
-            return false;
+    return TREATMENT_PATTERN.test(treatment);
+}
+function sanitizeGlobal(logger, treatment) {
+    if (treatment === undefined)
+        return undefined;
+    if (!isValidTreatment(treatment)) {
+        logger.error("Fallback treatments - Discarded fallback: " + FallbackDiscardReason.Treatment);
+        return undefined;
+    }
+    return treatment;
+}
+function sanitizeByFlag(logger, byFlagFallbacks) {
+    var sanitizedByFlag = {};
+    if (!isObject(byFlagFallbacks))
+        return sanitizedByFlag;
+    Object.keys(byFlagFallbacks).forEach(function (flag) {
+        var t = byFlagFallbacks[flag];
+        if (!isValidFlagName(flag)) {
+            logger.error("Fallback treatments - Discarded flag '" + flag + "': " + FallbackDiscardReason.FlagName);
+            return;
         }
-        return FallbacksSanitizer.pattern.test(treatment);
-    };
-    FallbacksSanitizer.sanitizeGlobal = function (logger, treatment) {
-        if (!this.isValidTreatment(treatment)) {
-            logger.error("Fallback treatments - Discarded fallback: " + FallbackDiscardReason.Treatment);
-            return undefined;
+        if (!isValidTreatment(t)) {
+            logger.error("Fallback treatments - Discarded treatment for flag '" + flag + "': " + FallbackDiscardReason.Treatment);
+            return;
         }
-        return treatment;
-    };
-    FallbacksSanitizer.sanitizeByFlag = function (logger, byFlagFallbacks) {
-        var _this = this;
-        var sanitizedByFlag = {};
-        var entries = Object.keys(byFlagFallbacks);
-        entries.forEach(function (flag) {
-            var t = byFlagFallbacks[flag];
-            if (!_this.isValidFlagName(flag)) {
-                logger.error("Fallback treatments - Discarded flag '" + flag + "': " + FallbackDiscardReason.FlagName);
-                return;
-            }
-            if (!_this.isValidTreatment(t)) {
-                logger.error("Fallback treatments - Discarded treatment for flag '" + flag + "': " + FallbackDiscardReason.Treatment);
-                return;
-            }
-            sanitizedByFlag[flag] = t;
-        });
-        return sanitizedByFlag;
+        sanitizedByFlag[flag] = t;
+    });
+    return sanitizedByFlag;
+}
+export function sanitizeFallbacks(logger, fallbacks) {
+    if (!isObject(fallbacks)) {
+        logger.error('Fallback treatments - Discarded configuration: it must be an object with optional `global` and `byFlag` properties');
+        return;
+    }
+    return {
+        global: sanitizeGlobal(logger, fallbacks.global),
+        byFlag: sanitizeByFlag(logger, fallbacks.byFlag)
     };
-    FallbacksSanitizer.pattern = /^[0-9]+[.a-zA-Z0-9_-]*$|^[a-zA-Z]+[a-zA-Z0-9_-]*$/;
-    return FallbacksSanitizer;
-}());
-export { FallbacksSanitizer };
+}

package/esm/evaluator/fallbackTreatmentsCalculator/index.js

@@ -1,15 +1,10 @@
-import { FallbacksSanitizer } from './fallbackSanitizer';
 import { CONTROL } from '../../utils/constants';
 import { isString } from '../../utils/lang';
 export var FALLBACK_PREFIX = 'fallback - ';
 var FallbackTreatmentsCalculator = /** @class */ (function () {
-    function FallbackTreatmentsCalculator(logger, fallbacks) {
-        var sanitizedGlobal = (fallbacks === null || fallbacks === void 0 ? void 0 : fallbacks.global) ? FallbacksSanitizer.sanitizeGlobal(logger, fallbacks.global) : undefined;
-        var sanitizedByFlag = (fallbacks === null || fallbacks === void 0 ? void 0 : fallbacks.byFlag) ? FallbacksSanitizer.sanitizeByFlag(logger, fallbacks.byFlag) : {};
-        this.fallbacks = {
-            global: sanitizedGlobal,
-            byFlag: sanitizedByFlag
-        };
+    function FallbackTreatmentsCalculator(fallbacks) {
+        if (fallbacks === void 0) { fallbacks = {}; }
+        this.fallbacks = fallbacks;
     }
     FallbackTreatmentsCalculator.prototype.resolve = function (flagName, label) {
         var _a;

package/esm/sdkFactory/index.js

@@ -49,7 +49,7 @@ export function sdkFactory(params) {
             readiness.splits.emit(SDK_SPLITS_CACHE_LOADED);
         }
     });
-    var fallbackTreatmentsCalculator = new FallbackTreatmentsCalculator(settings.log, settings.fallbackTreatments);
+    var fallbackTreatmentsCalculator = new FallbackTreatmentsCalculator(settings.fallbackTreatments);
     if (initialRolloutPlan) {
         setRolloutPlan(log, initialRolloutPlan, storage, key && getMatching(key));
         if (storage.splits.getChangeNumber() > -1)

package/esm/storages/AbstractMySegmentsCacheSync.js

@@ -20,37 +20,45 @@ var AbstractMySegmentsCacheSync = /** @class */ (function () {
      */
     AbstractMySegmentsCacheSync.prototype.resetSegments = function (segmentsData) {
         var _this = this;
-        this.setChangeNumber(segmentsData.cn);
+        var isDiff = false;
         var _a = segmentsData, added = _a.added, removed = _a.removed;
         if (added && removed) {
-            var isDiff_1 = false;
             added.forEach(function (segment) {
-                isDiff_1 = _this.addSegment(segment) || isDiff_1;
+                isDiff = _this.addSegment(segment) || isDiff;
             });
             removed.forEach(function (segment) {
-                isDiff_1 = _this.removeSegment(segment) || isDiff_1;
+                isDiff = _this.removeSegment(segment) || isDiff;
             });
-            return isDiff_1;
-        }
-        var names = (segmentsData.k || []).map(function (s) { return s.n; }).sort();
-        var storedSegmentKeys = this.getRegisteredSegments().sort();
-        // Extreme fast => everything is empty
-        if (!names.length && !storedSegmentKeys.length)
-            return false;
-        var index = 0;
-        while (index < names.length && index < storedSegmentKeys.length && names[index] === storedSegmentKeys[index])
-            index++;
-        // Quick path => no changes
-        if (index === names.length && index === storedSegmentKeys.length)
-            return false;
-        // Slowest path => add and/or remove segments
-        for (var removeIndex = index; removeIndex < storedSegmentKeys.length; removeIndex++) {
-            this.removeSegment(storedSegmentKeys[removeIndex]);
         }
-        for (var addIndex = index; addIndex < names.length; addIndex++) {
-            this.addSegment(names[addIndex]);
+        else {
+            var names = (segmentsData.k || []).map(function (s) { return s.n; }).sort();
+            var storedSegmentKeys = this.getRegisteredSegments().sort();
+            // Extreme fast => everything is empty
+            if (!names.length && !storedSegmentKeys.length) {
+                isDiff = false;
+            }
+            else {
+                var index = 0;
+                while (index < names.length && index < storedSegmentKeys.length && names[index] === storedSegmentKeys[index])
+                    index++;
+                // Quick path => no changes
+                if (index === names.length && index === storedSegmentKeys.length) {
+                    isDiff = false;
+                }
+                else {
+                    // Slowest path => add and/or remove segments
+                    for (var removeIndex = index; removeIndex < storedSegmentKeys.length; removeIndex++) {
+                        this.removeSegment(storedSegmentKeys[removeIndex]);
+                    }
+                    for (var addIndex = index; addIndex < names.length; addIndex++) {
+                        this.addSegment(names[addIndex]);
+                    }
+                    isDiff = true;
+                }
+            }
         }
-        return true;
+        this.setChangeNumber(segmentsData.cn);
+        return isDiff;
     };
     return AbstractMySegmentsCacheSync;
 }());

package/esm/storages/AbstractSplitsCacheSync.js

@@ -9,9 +9,10 @@ var AbstractSplitsCacheSync = /** @class */ (function () {
     }
     AbstractSplitsCacheSync.prototype.update = function (toAdd, toRemove, changeNumber) {
         var _this = this;
-        this.setChangeNumber(changeNumber);
         var updated = toAdd.map(function (addedFF) { return _this.addSplit(addedFF); }).some(function (result) { return result; });
-        return toRemove.map(function (removedFF) { return _this.removeSplit(removedFF.name); }).some(function (result) { return result; }) || updated;
+        updated = toRemove.map(function (removedFF) { return _this.removeSplit(removedFF.name); }).some(function (result) { return result; }) || updated;
+        this.setChangeNumber(changeNumber);
+        return updated;
     };
     AbstractSplitsCacheSync.prototype.getSplits = function (names) {
         var _this = this;

package/esm/storages/inLocalStorage/MySegmentsCacheInLocal.js

@@ -1,7 +1,7 @@
 import { __extends } from "tslib";
 import { isNaNNumber } from '../../utils/lang';
 import { AbstractMySegmentsCacheSync } from '../AbstractMySegmentsCacheSync';
-import { LOG_PREFIX, DEFINED } from './constants';
+import { DEFINED } from './constants';
 var MySegmentsCacheInLocal = /** @class */ (function (_super) {
     __extends(MySegmentsCacheInLocal, _super);
     function MySegmentsCacheInLocal(log, keys, storage) {
@@ -10,33 +10,20 @@ var MySegmentsCacheInLocal = /** @class */ (function (_super) {
         _this.keys = keys;
         _this.storage = storage;
         return _this;
-        // There is not need to flush segments cache like splits cache, since resetSegments receives the up-to-date list of active segments
     }
     MySegmentsCacheInLocal.prototype.addSegment = function (name) {
         var segmentKey = this.keys.buildSegmentNameKey(name);
-        try {
-            if (this.storage.getItem(segmentKey) === DEFINED)
-                return false;
-            this.storage.setItem(segmentKey, DEFINED);
-            return true;
-        }
-        catch (e) {
-            this.log.error(LOG_PREFIX + e);
+        if (this.storage.getItem(segmentKey) === DEFINED)
             return false;
-        }
+        this.storage.setItem(segmentKey, DEFINED);
+        return true;
     };
     MySegmentsCacheInLocal.prototype.removeSegment = function (name) {
         var segmentKey = this.keys.buildSegmentNameKey(name);
-        try {
-            if (this.storage.getItem(segmentKey) !== DEFINED)
-                return false;
-            this.storage.removeItem(segmentKey);
-            return true;
-        }
-        catch (e) {
-            this.log.error(LOG_PREFIX + e);
+        if (this.storage.getItem(segmentKey) !== DEFINED)
             return false;
-        }
+        this.storage.removeItem(segmentKey);
+        return true;
     };
     MySegmentsCacheInLocal.prototype.isInSegment = function (name) {
         return this.storage.getItem(this.keys.buildSegmentNameKey(name)) === DEFINED;
@@ -54,15 +41,10 @@ var MySegmentsCacheInLocal = /** @class */ (function (_super) {
         return 1;
     };
     MySegmentsCacheInLocal.prototype.setChangeNumber = function (changeNumber) {
-        try {
-            if (changeNumber)
-                this.storage.setItem(this.keys.buildTillKey(), changeNumber + '');
-            else
-                this.storage.removeItem(this.keys.buildTillKey());
-        }
-        catch (e) {
-            this.log.error(e);
-        }
+        if (changeNumber)
+            this.storage.setItem(this.keys.buildTillKey(), changeNumber + '');
+        else
+            this.storage.removeItem(this.keys.buildTillKey());
     };
     MySegmentsCacheInLocal.prototype.getChangeNumber = function () {
         var n = -1;

package/esm/storages/inLocalStorage/RBSegmentsCacheInLocal.js

@@ -15,9 +15,10 @@ var RBSegmentsCacheInLocal = /** @class */ (function () {
     };
     RBSegmentsCacheInLocal.prototype.update = function (toAdd, toRemove, changeNumber) {
         var _this = this;
-        this.setChangeNumber(changeNumber);
         var updated = toAdd.map(function (toAdd) { return _this.add(toAdd); }).some(function (result) { return result; });
-        return toRemove.map(function (toRemove) { return _this.remove(toRemove.name); }).some(function (result) { return result; }) || updated;
+        updated = toRemove.map(function (toRemove) { return _this.remove(toRemove.name); }).some(function (result) { return result; }) || updated;
+        this.setChangeNumber(changeNumber);
+        return updated;
     };
     RBSegmentsCacheInLocal.prototype.setChangeNumber = function (changeNumber) {
         try {
@@ -37,40 +38,28 @@ var RBSegmentsCacheInLocal = /** @class */ (function () {
             this.storage.removeItem(segmentsCountKey);
     };
     RBSegmentsCacheInLocal.prototype.add = function (rbSegment) {
-        try {
-            var name_1 = rbSegment.name;
-            var rbSegmentKey = this.keys.buildRBSegmentKey(name_1);
-            var rbSegmentFromStorage = this.storage.getItem(rbSegmentKey);
-            var previous = rbSegmentFromStorage ? JSON.parse(rbSegmentFromStorage) : null;
-            this.storage.setItem(rbSegmentKey, JSON.stringify(rbSegment));
-            var usesSegmentsDiff = 0;
-            if (previous && usesSegments(previous))
-                usesSegmentsDiff--;
-            if (usesSegments(rbSegment))
-                usesSegmentsDiff++;
-            if (usesSegmentsDiff !== 0)
-                this.updateSegmentCount(usesSegmentsDiff);
-            return true;
-        }
-        catch (e) {
-            this.log.error(LOG_PREFIX + e);
-            return false;
-        }
+        var name = rbSegment.name;
+        var rbSegmentKey = this.keys.buildRBSegmentKey(name);
+        var rbSegmentFromStorage = this.storage.getItem(rbSegmentKey);
+        var previous = rbSegmentFromStorage ? JSON.parse(rbSegmentFromStorage) : null;
+        this.storage.setItem(rbSegmentKey, JSON.stringify(rbSegment));
+        var usesSegmentsDiff = 0;
+        if (previous && usesSegments(previous))
+            usesSegmentsDiff--;
+        if (usesSegments(rbSegment))
+            usesSegmentsDiff++;
+        if (usesSegmentsDiff !== 0)
+            this.updateSegmentCount(usesSegmentsDiff);
+        return true;
     };
     RBSegmentsCacheInLocal.prototype.remove = function (name) {
-        try {
-            var rbSegment = this.get(name);
-            if (!rbSegment)
-                return false;
-            this.storage.removeItem(this.keys.buildRBSegmentKey(name));
-            if (usesSegments(rbSegment))
-                this.updateSegmentCount(-1);
-            return true;
-        }
-        catch (e) {
-            this.log.error(LOG_PREFIX + e);
+        var rbSegment = this.get(name);
+        if (!rbSegment)
             return false;
-        }
+        this.storage.removeItem(this.keys.buildRBSegmentKey(name));
+        if (usesSegments(rbSegment))
+            this.updateSegmentCount(-1);
+        return true;
     };
     RBSegmentsCacheInLocal.prototype.getNames = function () {
         var len = this.storage.length;

package/esm/storages/inLocalStorage/SplitsCacheInLocal.js

@@ -67,39 +67,27 @@ var SplitsCacheInLocal = /** @class */ (function (_super) {
         this.hasSync = false;
     };
     SplitsCacheInLocal.prototype.addSplit = function (split) {
-        try {
-            var name_1 = split.name;
-            var splitKey = this.keys.buildSplitKey(name_1);
-            var splitFromStorage = this.storage.getItem(splitKey);
-            var previousSplit = splitFromStorage ? JSON.parse(splitFromStorage) : null;
-            if (previousSplit) {
-                this._decrementCounts(previousSplit);
-                this.removeFromFlagSets(previousSplit.name, previousSplit.sets);
-            }
-            this.storage.setItem(splitKey, JSON.stringify(split));
-            this._incrementCounts(split);
-            this.addToFlagSets(split);
-            return true;
-        }
-        catch (e) {
-            this.log.error(LOG_PREFIX + e);
-            return false;
+        var name = split.name;
+        var splitKey = this.keys.buildSplitKey(name);
+        var splitFromStorage = this.storage.getItem(splitKey);
+        var previousSplit = splitFromStorage ? JSON.parse(splitFromStorage) : null;
+        if (previousSplit) {
+            this._decrementCounts(previousSplit);
+            this.removeFromFlagSets(previousSplit.name, previousSplit.sets);
         }
+        this.storage.setItem(splitKey, JSON.stringify(split));
+        this._incrementCounts(split);
+        this.addToFlagSets(split);
+        return true;
     };
     SplitsCacheInLocal.prototype.removeSplit = function (name) {
-        try {
-            var split = this.getSplit(name);
-            if (!split)
-                return false;
-            this.storage.removeItem(this.keys.buildSplitKey(name));
-            this._decrementCounts(split);
-            this.removeFromFlagSets(split.name, split.sets);
-            return true;
-        }
-        catch (e) {
-            this.log.error(LOG_PREFIX + e);
+        var split = this.getSplit(name);
+        if (!split)
             return false;
-        }
+        this.storage.removeItem(this.keys.buildSplitKey(name));
+        this._decrementCounts(split);
+        this.removeFromFlagSets(split.name, split.sets);
+        return true;
     };
     SplitsCacheInLocal.prototype.getSplit = function (name) {
         var item = this.storage.getItem(this.keys.buildSplitKey(name));
@@ -171,6 +159,8 @@ var SplitsCacheInLocal = /** @class */ (function (_super) {
             var flagSetKey = _this.keys.buildFlagSetKey(featureFlagSet);
             var flagSetFromStorage = _this.storage.getItem(flagSetKey);
             var flagSetCache = new Set(flagSetFromStorage ? JSON.parse(flagSetFromStorage) : []);
+            if (flagSetCache.has(featureFlag.name))
+                return;
             flagSetCache.add(featureFlag.name);
             _this.storage.setItem(flagSetKey, JSON.stringify(setToArray(flagSetCache)));
         });

package/esm/storages/inMemory/RBSegmentsCacheInMemory.js

@@ -13,9 +13,10 @@ var RBSegmentsCacheInMemory = /** @class */ (function () {
     };
     RBSegmentsCacheInMemory.prototype.update = function (toAdd, toRemove, changeNumber) {
         var _this = this;
-        this.changeNumber = changeNumber;
         var updated = toAdd.map(function (toAdd) { return _this.add(toAdd); }).some(function (result) { return result; });
-        return toRemove.map(function (toRemove) { return _this.remove(toRemove.name); }).some(function (result) { return result; }) || updated;
+        updated = toRemove.map(function (toRemove) { return _this.remove(toRemove.name); }).some(function (result) { return result; }) || updated;
+        this.changeNumber = changeNumber;
+        return updated;
     };
     RBSegmentsCacheInMemory.prototype.add = function (rbSegment) {
         var name = rbSegment.name;

package/esm/storages/inRedis/SegmentsCacheInRedis.js

@@ -1,5 +1,5 @@
 import { isNaNNumber } from '../../utils/lang';
-import { LOG_PREFIX } from '../inLocalStorage/constants';
+import { LOG_PREFIX } from './constants';
 var SegmentsCacheInRedis = /** @class */ (function () {
     function SegmentsCacheInRedis(log, keys, redis) {
         this.log = log;

package/esm/sync/polling/syncTasks/segmentsSyncTask.js

@@ -5,5 +5,5 @@ import { segmentChangesUpdaterFactory } from '../updaters/segmentChangesUpdater'
  * Creates a sync task that periodically executes a `segmentChangesUpdater` task
  */
 export function segmentsSyncTaskFactory(fetchSegmentChanges, storage, readiness, settings) {
-    return syncTaskFactory(settings.log, segmentChangesUpdaterFactory(settings.log, segmentChangesFetcherFactory(fetchSegmentChanges), storage.segments, readiness), settings.scheduler.segmentsRefreshRate, 'segmentChangesUpdater');
+    return syncTaskFactory(settings.log, segmentChangesUpdaterFactory(settings.log, segmentChangesFetcherFactory(fetchSegmentChanges), storage.segments, readiness, settings.startup.requestTimeoutBeforeReady, settings.startup.retriesOnFailureBeforeReady), settings.scheduler.segmentsRefreshRate, 'segmentChangesUpdater');
 }

package/esm/sync/polling/updaters/mySegmentsUpdater.js

@@ -45,9 +45,9 @@ export function mySegmentsUpdaterFactory(log, mySegmentsFetcher, storage, segmen
             new Promise(function (res) { updateSegments(segmentsData); res(true); }) :
             // If not provided, fetch mySegments
             mySegmentsFetcher(matchingKey, noCache, till, _promiseDecorator).then(function (segments) {
-                // Only when we have downloaded segments completely, we should not keep retrying anymore
-                startingUp = false;
                 updateSegments(segments);
+                // Only when we have downloaded and stored segments completely, we should not keep retrying anymore
+                startingUp = false;
                 return true;
             });
         return updaterPromise.catch(function (error) {

package/esm/sync/polling/updaters/segmentChangesUpdater.js

@@ -1,5 +1,6 @@
 import { SDK_SEGMENTS_ARRIVED } from '../../../readiness/constants';
 import { LOG_PREFIX_INSTANTIATION, LOG_PREFIX_SYNC_SEGMENTS } from '../../../logger/constants';
+import { timeout } from '../../../utils/promise/timeout';
 /**
  * Factory of SegmentChanges updater, a task that:
  *  - fetches segment changes using `segmentChangesFetcher`
@@ -11,22 +12,33 @@ import { LOG_PREFIX_INSTANTIATION, LOG_PREFIX_SYNC_SEGMENTS } from '../../../log
  * @param segments - segments storage, with sync or async methods
  * @param readiness - optional readiness manager. Not required for synchronizer or producer mode.
  */
-export function segmentChangesUpdaterFactory(log, segmentChangesFetcher, segments, readiness) {
+export function segmentChangesUpdaterFactory(log, segmentChangesFetcher, segments, readiness, requestTimeoutBeforeReady, retriesOnFailureBeforeReady) {
     var readyOnAlreadyExistentState = true;
-    function updateSegment(segmentName, noCache, till, fetchOnlyNew) {
+    function _promiseDecorator(promise) {
+        if (readyOnAlreadyExistentState && requestTimeoutBeforeReady)
+            promise = timeout(requestTimeoutBeforeReady, promise);
+        return promise;
+    }
+    function updateSegment(segmentName, noCache, till, fetchOnlyNew, retries) {
         log.debug(LOG_PREFIX_SYNC_SEGMENTS + "Processing segment " + segmentName);
         var sincePromise = Promise.resolve(segments.getChangeNumber(segmentName));
         return sincePromise.then(function (since) {
             // if fetchOnlyNew flag, avoid processing already fetched segments
             return fetchOnlyNew && since !== undefined ?
                 false :
-                segmentChangesFetcher(since || -1, segmentName, noCache, till).then(function (changes) {
+                segmentChangesFetcher(since || -1, segmentName, noCache, till, _promiseDecorator).then(function (changes) {
                     return Promise.all(changes.map(function (x) {
                         log.debug(LOG_PREFIX_SYNC_SEGMENTS + "Processing " + segmentName + " with till = " + x.till + ". Added: " + x.added.length + ". Removed: " + x.removed.length);
                         return segments.update(segmentName, x.added, x.removed, x.till);
                     })).then(function (updates) {
                         return updates.some(function (update) { return update; });
                     });
+                }).catch(function (error) {
+                    if (retries) {
+                        log.warn(LOG_PREFIX_SYNC_SEGMENTS + "Retrying fetch of segment " + segmentName + " (attempt #" + retries + "). Reason: " + error);
+                        return updateSegment(segmentName, noCache, till, fetchOnlyNew, retries - 1);
+                    }
+                    throw error;
                 });
         });
     }
@@ -46,8 +58,7 @@ export function segmentChangesUpdaterFactory(log, segmentChangesFetcher, segment
         // If not a segment name provided, read list of available segments names to be updated.
         var segmentsPromise = Promise.resolve(segmentName ? [segmentName] : segments.getRegisteredSegments());
         return segmentsPromise.then(function (segmentNames) {
-            // Async fetchers
-            var updaters = segmentNames.map(function (segmentName) { return updateSegment(segmentName, noCache, till, fetchOnlyNew); });
+            var updaters = segmentNames.map(function (segmentName) { return updateSegment(segmentName, noCache, till, fetchOnlyNew, readyOnAlreadyExistentState ? retriesOnFailureBeforeReady : 0); });
             return Promise.all(updaters).then(function (shouldUpdateFlags) {
                 // if at least one segment fetch succeeded, mark segments ready
                 if (shouldUpdateFlags.some(function (update) { return update; }) || readyOnAlreadyExistentState) {

package/esm/sync/polling/updaters/splitChangesUpdater.js

@@ -130,7 +130,6 @@ export function splitChangesUpdaterFactory(log, splitChangesFetcher, storage, sp
                     { rbs: { d: [instantUpdate.payload], t: instantUpdate.changeNumber } } :
                 splitChangesFetcher(since, noCache, till, rbSince, _promiseDecorator))
                 .then(function (splitChanges) {
-                startingUp = false;
                 var usedSegments = new Set();
                 var ffUpdate = false;
                 if (splitChanges.ff) {
@@ -151,6 +150,7 @@ export function splitChangesUpdaterFactory(log, splitChangesFetcher, storage, sp
                     var ffChanged = _a[0], rbsChanged = _a[1];
                     if (storage.save)
                         storage.save();
+                    startingUp = false;
                     if (splitsEventEmitter) {
                         // To emit SDK_SPLITS_ARRIVED for server-side SDK, we must check that all registered segments have been fetched
                         return Promise.resolve(!splitsEventEmitter.splitsArrived || ((ffChanged || rbsChanged) && (isClientSide || checkAllSegmentsExist(segments))))
@@ -166,14 +166,14 @@ export function splitChangesUpdaterFactory(log, splitChangesFetcher, storage, sp
                 });
             })
                 .catch(function (error) {
-                log.warn(SYNC_SPLITS_FETCH_FAILS, [error]);
                 if (startingUp && retriesOnFailureBeforeReady > retry) {
                     retry += 1;
-                    log.info(SYNC_SPLITS_FETCH_RETRY, [retry, error]);
+                    log.warn(SYNC_SPLITS_FETCH_RETRY, [retry, error]);
                     return _splitChangesUpdater(sinces, retry);
                 }
                 else {
                     startingUp = false;
+                    log.warn(SYNC_SPLITS_FETCH_FAILS, [error]);
                 }
                 return false;
             });

package/esm/utils/settingsValidation/index.js

@@ -6,6 +6,7 @@ import { validImpressionsMode } from './impressionsMode';
 import { validateKey } from '../inputValidation/key';
 import { ERROR_MIN_CONFIG_PARAM, LOG_PREFIX_CLIENT_INSTANTIATION } from '../../logger/constants';
 import { validateRolloutPlan } from '../../storages/setRolloutPlan';
+import { sanitizeFallbacks } from '../../evaluator/fallbackTreatmentsCalculator/fallbackSanitizer';
 // Exported for telemetry
 export var base = {
     // Define which kind of object you want to retrieve from SplitFactory
@@ -180,5 +181,7 @@ export function settingsValidation(config, validationParams) {
     // ensure a valid user consent value
     // @ts-ignore, modify readonly prop
     withDefaults.userConsent = consent ? consent(withDefaults) : undefined;
+    // @ts-ignore, modify readonly prop
+    withDefaults.fallbackTreatments = withDefaults.fallbackTreatments ? sanitizeFallbacks(log, withDefaults.fallbackTreatments) : undefined;
     return withDefaults;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@splitsoftware/splitio-commons",
-  "version": "2.7.9-rc.3",
+  "version": "2.8.1-rc.0",
   "description": "Split JavaScript SDK common components",
   "main": "cjs/index.js",
   "module": "esm/index.js",