@splitsoftware/splitio 10.15.10-rc.2 → 10.16.1-rc.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@splitsoftware/splitio",
-  "version": "10.15.10-rc.2",
+  "version": "10.16.1-rc.0",
   "description": "Split SDK",
   "files": [
     "README.md",
@@ -35,7 +35,7 @@
     "@babel/runtime": "^7.13.10",
     "@types/google.analytics": "0.0.40",
     "events": "3.1.0",
-    "ioredis": "^4.26.0",
+    "ioredis": "^4.28.0",
     "ip": "1.1.5",
     "js-yaml": "3.13.1",
     "node-fetch": "^2.6.1",
@@ -61,8 +61,9 @@
     "karma": "^6.3.2",
     "karma-chrome-launcher": "^3.1.0",
     "karma-tap": "4.1.4",
-    "karma-webpack": "^4.0.2",
+    "karma-webpack": "^5.0.0",
     "lodash": "^4.17.21",
+    "node-polyfill-webpack-plugin": "^1.1.4",
     "proxyquire": "^2.1.3",
     "puppeteer": "^3.3.0",
     "redis-dumpz": "0.1.11",
@@ -73,9 +74,9 @@
     "tap-min": "^2.0.0",
     "tape": "4.13.2",
     "tape-catch": "1.0.6",
-    "webpack": "^4.46.0",
-    "webpack-cli": "^3.3.12",
-    "webpack-merge": "^4.2.2"
+    "webpack": "^5.53.0",
+    "webpack-cli": "^4.8.0",
+    "webpack-merge": "^5.8.0"
   },
   "scripts": {
     "build-es": "rimraf es && cross-env NODE_ENV=es babel ./src -d es --ignore 'src/**/__tests__/*'",
@@ -83,25 +84,27 @@
     "build-umd:stats": "webpack --progress --env production --json > ./stats/stat_results.json",
     "build-cjs": "rimraf lib && cross-env NODE_ENV=cjs babel ./src -d lib --ignore 'src/**/__tests__/*'",
     "postbuild-cjs": "cross-env NODE_ENV=cjs node scripts/copy.packages.json.js",
-    "build-umd:ci": "webpack --config webpack.ci.dev.js --env.branch=$BUILD_BRANCH --env.commit_hash=$BUILD_COMMIT && webpack --config webpack.ci.prod.js --env.branch=$BUILD_BRANCH --env.commit_hash=$BUILD_COMMIT && ./scripts/clean_umd_build.sh",
-    "build-umd": "webpack --config webpack.dev.js && webpack --config webpack.prod.js && ./scripts/clean_umd_build.sh",
-    "build": "rimraf lib umd es && npm run build-cjs && npm run build-es && npm run build-umd",
+    "build-umd": "webpack --config webpack.ci.dev.js --env branch=$BUILD_BRANCH --env commit_hash=$BUILD_COMMIT && webpack --config webpack.ci.prod.js --env branch=$BUILD_BRANCH --env commit_hash=$BUILD_COMMIT && ./scripts/clean_umd_build.sh",
+    "build-umd:local": "webpack --config webpack.dev.js && webpack --config webpack.prod.js && ./scripts/clean_umd_build.sh",
+    "build:local": "rimraf lib umd es && npm run build-cjs && npm run build-es && npm run build-umd:local",
     "build:npm": "rimraf lib es && npm run build-cjs && npm run build-es",
-    "build:ci": "rimraf lib umd es && npm run build-cjs && npm run build-es && npm run build-umd:ci",
-    "check": "npm run check:lint && npm run check:version ",
+    "build": "rimraf lib umd es && npm run build-cjs && npm run build-es && npm run build-umd",
+    "check": "npm run check:lint && npm run check:version",
     "check:lint": "eslint src",
     "check:version": "cross-env NODE_ENV=test tape -r @babel/register src/utils/__tests__/settings/index.spec.js",
     "test-browser-local": "cross-env NODE_ENV=test karma start karma/local.karma.conf.js",
     "test-browser-e2e-local": "cross-env NODE_ENV=test karma start karma/e2e.local.karma.conf.js",
-    "test-browser-ci": "cross-env NODE_ENV=test karma start karma/ci.karma.conf.js",
-    "test-browser-e2e-ci": "cross-env NODE_ENV=test karma start karma/e2e.ci.karma.conf.js",
+    "test-browser": "npm run test-browser-unit && npm run test-browser-online && npm run test-browser-offline && npm run test-browser-destroy && npm run test-browser-errors && npm run test-browser-push",
+    "test-browser-unit": "cross-env NODE_ENV=test karma start karma/ci.karma.conf.js",
+    "test-browser-online": "cross-env NODE_ENV=test karma start karma/e2e.ci.karma.conf.js",
     "test-browser-offline": "cross-env NODE_ENV=test karma start karma/offline.karma.conf.js",
     "test-browser-destroy": "cross-env NODE_ENV=test karma start karma/destroy.ci.karma.conf.js",
     "test-browser-errors": "cross-env NODE_ENV=test karma start karma/errors.ci.karma.conf.js",
     "test-browser-gaintegration": "cross-env NODE_ENV=test karma start karma/gaintegration.ci.karma.conf.js",
     "test-browser-push": "cross-env NODE_ENV=test karma start karma/push.ci.karma.conf.js",
-    "test-node": "cross-env NODE_ENV=test tape -r @babel/register \"src/*/**/__tests__/**/!(browser).spec.js\" | tap-min",
-    "test-node-e2e": "cross-env NODE_ENV=test tape -r @babel/register src/__tests__/node.spec.js | tap-min",
+    "test-node": "npm run test-node-unit && npm run test-node-online && npm run test-node-redis && npm run test-node-offline && npm run test-node-destroy && npm run test-node-errors && npm run test-node-push",
+    "test-node-unit": "cross-env NODE_ENV=test tape -r @babel/register \"src/*/**/__tests__/**/!(browser).spec.js\" | tap-min",
+    "test-node-online": "cross-env NODE_ENV=test tape -r @babel/register src/__tests__/node.spec.js | tap-min",
     "test-node-destroy": "cross-env NODE_ENV=test tape -r @babel/register src/__tests__/destroy/node.spec.js | tap-min",
     "test-node-errors": "cross-env NODE_ENV=test tape -r @babel/register src/__tests__/errorCatching/node.spec.js | tap-min",
     "test-node-offline": "cross-env NODE_ENV=test tape -r @babel/register src/__tests__/offline/node.spec.js | tap-min",

package/src/engine/engine/murmur3/murmur3_128.js

@@ -265,7 +265,7 @@ function hash128x64(key /*: string */, seed /*: number */) /*: string */ {
 
 /**
  * x64 version of Murmur3 for 128bits.
- * Used in hashImpression128.
+ *
  * @param {string} str
  */
 export function hash128(str /*: string */, seed /*: number */) /*: string */ {

package/src/engine/engine/murmur3/murmur3_128_x86.js

@@ -179,10 +179,9 @@ function hash128x86(key /*: string */, seed /*: number */) /*: string */ {
 
 /**
  * x86 version of Murmur3 for 128bits.
- * @param {string} str
+ * Used by hashImpression128 because in JS it is more efficient than the x64 version, no matter the underlying OS/CPU arch.
  *
- * @TODO Test and use in hashImpression128 instead of the x64 version,
- * because it is more time efficient in JS, no matter the underlying CPU arch.
+ * @param {string} str
  */
 export function hash128(str /*: string */, seed /*: number */) /*: string */ {
   return hash128x86(UTF16ToUTF8(str), seed >>> 0);

package/src/engine/matchers/index.js

@@ -13,7 +13,6 @@ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 See the License for the specific language governing permissions and
 limitations under the License.
 **/
-import { types } from './types';
 import allMatcher from './all';
 import segmentMatcher from './segment';
 import whitelistMatcher from './whitelist';
@@ -32,6 +31,27 @@ import dependencyMatcher from './dependency';
 import booleanMatcher from './boolean';
 import stringMatcher from './string';
 
+const matchers = [
+  undefined, // UNDEFINED: 0,
+  allMatcher, // ALL_KEYS: 1,
+  segmentMatcher, // IN_SEGMENT: 2,
+  whitelistMatcher, // WHITELIST: 3,
+  eqMatcher, // EQUAL_TO: 4,
+  gteMatcher, // GREATER_THAN_OR_EQUAL_TO: 5,
+  lteMatcher, // LESS_THAN_OR_EQUAL_TO: 6,
+  betweenMatcher, // BETWEEN: 7,
+  equalToSetMatcher, // EQUAL_TO_SET: 8,
+  containsAnySetMatcher, // CONTAINS_ANY_OF_SET: 9,
+  containsAllSetMatcher, // CONTAINS_ALL_OF_SET: 10,
+  partOfSetMatcher, // PART_OF_SET: 11,
+  ewMatcher, // ENDS_WITH: 12,
+  swMatcher, // STARTS_WITH: 13,
+  containsStrMatcher, // CONTAINS_STRING: 14,
+  dependencyMatcher, // IN_SPLIT_TREATMENT: 15,
+  booleanMatcher, // EQUAL_TO_BOOLEAN: 16,
+  stringMatcher // MATCHES_STRING: 17
+];
+
 /**
  * Matcher factory.
  */
@@ -43,41 +63,7 @@ function MatcherFactory(matcherDto, storage) {
 
   let matcherFn;
 
-  if (type === types.ALL) {
-    matcherFn = allMatcher(value);
-  } else if (type === types.SEGMENT) {
-    matcherFn = segmentMatcher(value, storage);
-  } else if (type === types.WHITELIST) {
-    matcherFn = whitelistMatcher(value);
-  } else if (type === types.EQUAL_TO) {
-    matcherFn = eqMatcher(value);
-  } else if (type === types.GREATER_THAN_OR_EQUAL_TO) {
-    matcherFn = gteMatcher(value);
-  } else if (type === types.LESS_THAN_OR_EQUAL_TO) {
-    matcherFn = lteMatcher(value);
-  } else if (type === types.BETWEEN) {
-    matcherFn = betweenMatcher(value);
-  } else if (type === types.EQUAL_TO_SET) {
-    matcherFn = equalToSetMatcher(value);
-  } else if (type === types.CONTAINS_ANY_OF_SET) {
-    matcherFn = containsAnySetMatcher(value);
-  } else if (type === types.CONTAINS_ALL_OF_SET) {
-    matcherFn = containsAllSetMatcher(value);
-  } else if (type === types.PART_OF_SET) {
-    matcherFn = partOfSetMatcher(value);
-  } else if (type === types.STARTS_WITH) {
-    matcherFn = swMatcher(value);
-  } else if (type === types.ENDS_WITH) {
-    matcherFn = ewMatcher(value);
-  } else if (type === types.CONTAINS_STRING) {
-    matcherFn = containsStrMatcher(value);
-  } else if (type === types.IN_SPLIT_TREATMENT) {
-    matcherFn = dependencyMatcher(value, storage);
-  } else if (type === types.EQUAL_TO_BOOLEAN) {
-    matcherFn = booleanMatcher(value);
-  } else if (type === types.MATCHES_STRING) {
-    matcherFn = stringMatcher(value);
-  }
+  if (matchers[type]) matcherFn = matchers[type](value, storage); // There is no index-out-of-bound exception in JavaScript
 
   return matcherFn;
 }

package/src/engine/matchers/types.js

@@ -15,28 +15,28 @@ limitations under the License.
 **/
 
 // @WARNING Symbol is not correctly working in PhantomJS
-export const types = {
-  ALL: 1,
-  SEGMENT: 2,
+export const matcherTypes = {
+  UNDEFINED: 0,
+  ALL_KEYS: 1,
+  IN_SEGMENT: 2,
   WHITELIST: 3,
   EQUAL_TO: 4,
   GREATER_THAN_OR_EQUAL_TO: 5,
   LESS_THAN_OR_EQUAL_TO: 6,
   BETWEEN: 7,
-  UNDEFINED: 8,
-  EQUAL_TO_SET: 9,
-  CONTAINS_ANY_OF_SET: 10,
-  CONTAINS_ALL_OF_SET: 11,
-  PART_OF_SET: 12,
-  ENDS_WITH: 13,
-  STARTS_WITH: 14,
-  CONTAINS_STRING: 15,
-  IN_SPLIT_TREATMENT: 16,
-  EQUAL_TO_BOOLEAN: 17,
-  MATCHES_STRING: 18
+  EQUAL_TO_SET: 8,
+  CONTAINS_ANY_OF_SET: 9,
+  CONTAINS_ALL_OF_SET: 10,
+  PART_OF_SET: 11,
+  ENDS_WITH: 12,
+  STARTS_WITH: 13,
+  CONTAINS_STRING: 14,
+  IN_SPLIT_TREATMENT: 15,
+  EQUAL_TO_BOOLEAN: 16,
+  MATCHES_STRING: 17
 };
 
-export const dataTypes = {
+export const matcherDataTypes = {
   BOOLEAN: 'BOOLEAN',
   STRING: 'STRING',
   NUMBER: 'NUMBER',
@@ -45,43 +45,8 @@ export const dataTypes = {
   NOT_SPECIFIED: 'NOT_SPECIFIED'
 };
 
-export const mapper = (matcherType) => {
-  switch (matcherType) {
-    case 'ALL_KEYS':
-      return types.ALL;
-    case 'IN_SEGMENT':
-      return types.SEGMENT;
-    case 'WHITELIST':
-      return types.WHITELIST;
-    case 'EQUAL_TO':
-      return types.EQUAL_TO;
-    case 'GREATER_THAN_OR_EQUAL_TO':
-      return types.GREATER_THAN_OR_EQUAL_TO;
-    case 'LESS_THAN_OR_EQUAL_TO':
-      return types.LESS_THAN_OR_EQUAL_TO;
-    case 'BETWEEN':
-      return types.BETWEEN;
-    case 'EQUAL_TO_SET':
-      return types.EQUAL_TO_SET;
-    case 'CONTAINS_ANY_OF_SET':
-      return types.CONTAINS_ANY_OF_SET;
-    case 'CONTAINS_ALL_OF_SET':
-      return types.CONTAINS_ALL_OF_SET;
-    case 'PART_OF_SET':
-      return types.PART_OF_SET;
-    case 'ENDS_WITH':
-      return types.ENDS_WITH;
-    case 'STARTS_WITH':
-      return types.STARTS_WITH;
-    case 'CONTAINS_STRING':
-      return types.CONTAINS_STRING;
-    case 'IN_SPLIT_TREATMENT':
-      return types.IN_SPLIT_TREATMENT;
-    case 'EQUAL_TO_BOOLEAN':
-      return types.EQUAL_TO_BOOLEAN;
-    case 'MATCHES_STRING':
-      return types.MATCHES_STRING;
-    default:
-      return types.UNDEFINED;
-  }
-};
+export function matcherTypesMapper(matcherType) {
+  const type = matcherTypes[matcherType];
+  if (type) return type;
+  else return matcherTypes.UNDEFINED;
+}

package/src/engine/transforms/matcherGroup.js

@@ -13,10 +13,7 @@ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 See the License for the specific language governing permissions and
 limitations under the License.
 **/
-import {
-  types as matcherTypes,
-  mapper as matcherTypesMapper
-} from '../matchers/types';
+import { matcherTypes, matcherTypesMapper } from '../matchers/types';
 import segmentTransform from './segment';
 import whitelistTransform from './whitelist';
 
@@ -33,7 +30,7 @@ function transform(matcherGroup) {
   let type = matcherTypesMapper(matcherType);
   let value = undefined;
 
-  if (type === matcherTypes.SEGMENT) {
+  if (type === matcherTypes.IN_SEGMENT) {
     value = segmentTransform(segmentObject);
   } else if (type === matcherTypes.WHITELIST) {
     value = whitelistTransform(whitelistObject);

package/src/engine/transforms/matchers.js

@@ -15,11 +15,7 @@ limitations under the License.
 **/
 
 import { findIndex } from '../../utils/lang';
-import {
-  types as matcherTypes,
-  mapper as matcherTypesMapper,
-  dataTypes as matcherDataTypes
-} from '../matchers/types';
+import { matcherTypes, matcherTypesMapper, matcherDataTypes } from '../matchers/types';
 import segmentTransform from './segment';
 import whitelistTransform from './whitelist';
 import setTransform from './set';
@@ -49,7 +45,7 @@ function transform(matchers) {
     let dataType = matcherDataTypes.STRING;
     let value = undefined;
 
-    if (type === matcherTypes.SEGMENT) {
+    if (type === matcherTypes.IN_SEGMENT) {
       value = segmentTransform(segmentObject);
     } else if (type === matcherTypes.WHITELIST) {
       value = whitelistTransform(whitelistObject);

package/src/engine/value/sanitize.js

@@ -18,12 +18,7 @@ import logFactory from '../../utils/logger';
 const log = logFactory('splitio-engine:sanitize');
 import { isObject, uniq, toString, toNumber } from '../../utils/lang';
 import { zeroSinceHH, zeroSinceSS } from '../convertions';
-import {
-  types as matcherTypes,
-  dataTypes as matcherDataTypes
-} from '../matchers/types';
-const MATCHERS = matcherTypes;
-const DATA_TYPES = matcherDataTypes;
+import { matcherTypes, matcherDataTypes } from '../matchers/types';
 
 function sanitizeNumber(val) {
   const num = toNumber(val);
@@ -72,13 +67,13 @@ function dependencyProcessor(sanitizedValue, attributes) {
  */
 function getProcessingFunction(matcherTypeID, dataType) {
   switch (matcherTypeID) {
-    case MATCHERS.EQUAL_TO:
+    case matcherTypes.EQUAL_TO:
       return dataType === 'DATETIME' ? zeroSinceHH : undefined;
-    case MATCHERS.GREATER_THAN_OR_EQUAL_TO:
-    case MATCHERS.LESS_THAN_OR_EQUAL_TO:
-    case MATCHERS.BETWEEN:
+    case matcherTypes.GREATER_THAN_OR_EQUAL_TO:
+    case matcherTypes.LESS_THAN_OR_EQUAL_TO:
+    case matcherTypes.BETWEEN:
       return dataType === 'DATETIME' ? zeroSinceSS : undefined;
-    case MATCHERS.IN_SPLIT_TREATMENT:
+    case matcherTypes.IN_SPLIT_TREATMENT:
       return dependencyProcessor;
     default:
       return undefined;
@@ -90,20 +85,20 @@ function sanitizeValue(matcherTypeID, value, dataType, attributes) {
   let sanitizedValue;
 
   switch (dataType) {
-    case DATA_TYPES.NUMBER:
-    case DATA_TYPES.DATETIME:
+    case matcherDataTypes.NUMBER:
+    case matcherDataTypes.DATETIME:
       sanitizedValue = sanitizeNumber(value);
       break;
-    case DATA_TYPES.STRING:
+    case matcherDataTypes.STRING:
       sanitizedValue = sanitizeString(value);
       break;
-    case DATA_TYPES.SET:
+    case matcherDataTypes.SET:
       sanitizedValue = sanitizeArray(value);
       break;
-    case DATA_TYPES.BOOLEAN:
+    case matcherDataTypes.BOOLEAN:
       sanitizedValue = sanitizeBoolean(value);
       break;
-    case DATA_TYPES.NOT_SPECIFIED:
+    case matcherDataTypes.NOT_SPECIFIED:
       sanitizedValue = value;
       break;
     default:

package/src/impressions/hasher/hashImpression128.js

@@ -1,4 +1,4 @@
-import { hash128 } from '../../engine/engine/murmur3/murmur3_128';
+import { hash128 } from '../../engine/engine/murmur3/murmur3_128_x86';
 import { buildKey } from './buildKey';
 
 export function hashImpression128(impression) {

package/src/impressions/observer/observer.js

@@ -7,8 +7,6 @@ class ImpressionObserver {
   }
 
   testAndSet(impression) {
-    if (!impression) return null;
-
     const hash = this.hasher(impression);
     const previous = this.cache.get(hash);
     this.cache.set(hash, impression.time);

package/src/listeners/browser.js

@@ -10,16 +10,8 @@ import objectAssign from 'object-assign';
 
 const log = logFactory('splitio-client:cleanup');
 
-function getUnloadDomEvent() {
-  // Except for Firefox (See https://github.com/splitio/react-client/issues/71), 'unload' event is used instead of 'beforeunload',
-  // since it is not a cancelable event and no other listeners can stop the event from occurring.
-  if (typeof navigator == 'object' && navigator.userAgent && navigator.userAgent.indexOf('Firefox') !== -1) {
-    return 'beforeunload';
-  }
-  return 'unload';
-}
-
-const UNLOAD_DOM_EVENT = getUnloadDomEvent();
+// 'unload' event is used instead of 'beforeunload', since 'unload' is not a cancelable event, so no other listeners can stop the event from occurring.
+const UNLOAD_DOM_EVENT = 'unload';
 
 /**
  * We'll listen for 'unload' event over the window object, since it's the standard way to listen page reload and close.

package/src/producer/updater/SplitChangesFromObject.js

@@ -13,6 +13,7 @@ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 See the License for the specific language governing permissions and
 limitations under the License.
 **/
+import { STORAGE_LOCALSTORAGE } from '../../utils/constants';
 import { forOwn } from '../../utils/lang';
 import logFactory from '../../utils/logger';
 const log = logFactory('splitio-producer:offline');
@@ -59,7 +60,6 @@ function FromObjectUpdaterFactory(Fetcher, context) {
 
       return Promise.all([
         storage.splits.flush(), // required to sync removed splits from mock
-        storage.splits.setChangeNumber(Date.now()),
         storage.splits.addSplits(splits)
       ]).then(() => {
         readiness.splits.emit(readiness.splits.SDK_SPLITS_ARRIVED);
@@ -67,7 +67,7 @@ function FromObjectUpdaterFactory(Fetcher, context) {
         if (startingUp) {
           startingUp = false;
           // Emits SDK_READY_FROM_CACHE
-          if (storage.splits.checkCache()) readiness.splits.emit(readiness.splits.SDK_SPLITS_CACHE_LOADED);
+          if (settings.storage.__originalType === STORAGE_LOCALSTORAGE) readiness.splits.emit(readiness.splits.SDK_SPLITS_CACHE_LOADED);
           // Only emits SDK_SEGMENTS_ARRIVED the first time for SDK_READY
           readiness.segments.emit(readiness.segments.SDK_SEGMENTS_ARRIVED);
         }

package/src/storage/SplitCache/InLocalStorage.js

@@ -11,7 +11,7 @@ class SplitCacheLocalStorage {
    * @param {number} expirationTimestamp
    * @param {Object} splitFiltersValidation
    */
-  constructor(keys, expirationTimestamp, splitFiltersValidation) {
+  constructor(keys, expirationTimestamp, splitFiltersValidation = { validFilters: [], queryString: null, groupedFilters: { byName: [], byPrefix: [] } }) {
     this.keys = keys;
     this.splitFiltersValidation = splitFiltersValidation;
 
@@ -129,13 +129,13 @@ class SplitCacheLocalStorage {
 
   setChangeNumber(changeNumber) {
     // when cache is ready but using a new split query, we must flush all split data
-    if(this.cacheReadyButNeedsToFlush) {
+    if (this.cacheReadyButNeedsToFlush) {
       this.flush();
       this.cacheReadyButNeedsToFlush = false;
     }
 
     // when using a new split query, we must update it at the store
-    if(this.updateNewFilter) {
+    if (this.updateNewFilter) {
       log.info('Split filter query was modified. Updating cache.');
       const queryKey = this.keys.buildSplitsFilterQueryKey();
       const { queryString } = this.splitFiltersValidation;

package/src/sync/PushManager/index.js

@@ -73,8 +73,8 @@ export default function PushManagerFactory(context, clientContexts /* undefined
     // Set token refresh 10 minutes before expirationTime - issuedAt
     const decodedToken = authData.decodedToken;
     const refreshTokenDelay = decodedToken.exp - decodedToken.iat - SECONDS_BEFORE_EXPIRATION;
-    // connDelay is present in AuthV2 but not in AuthV1
-    const connDelay = authData.connDelay || 0;
+    // Default connDelay of 60 secs
+    const connDelay = typeof authData.connDelay === 'number' && authData.connDelay >= 0 ? authData.connDelay : 60;
 
     log.info(`Refreshing streaming token in ${refreshTokenDelay} seconds, and connecting streaming in ${connDelay} seconds`);
 
@@ -247,19 +247,12 @@ export default function PushManagerFactory(context, clientContexts /* undefined
           }
 
           forOwn(clients, ({ hash64, worker }) => {
-            if (added.has(hash64.dec)) {
+            const add = added.has(hash64.dec) ? true : removed.has(hash64.dec) ? false : undefined;
+            if (add !== undefined) {
               worker.put(parsedData.changeNumber, {
                 name: parsedData.segmentName,
-                add: true
+                add
               });
-              return;
-            }
-            if (removed.has(hash64.dec)) {
-              worker.put(parsedData.changeNumber, {
-                name: parsedData.segmentName,
-                add: false
-              });
-              return;
             }
           });
           return;

package/src/sync/constants.js

@@ -40,4 +40,4 @@ export const ControlTypes = {
   STREAMING_PAUSED: 'STREAMING_PAUSED',
   STREAMING_RESUMED: 'STREAMING_RESUMED',
   STREAMING_RESET: 'STREAMING_RESET'
-};
+};

package/src/utils/inputValidation/splitFilters.js

@@ -69,7 +69,7 @@ export default function validateSplitFilters(splitFilters, mode) {
   const res = {
     validFilters: [],
     queryString: null,
-    groupedFilters: {}
+    groupedFilters: { byName: [], byPrefix: [] }
   };
   // do nothing if `splitFilters` param is not a non-empty array or mode is not STANDALONE
   if (!splitFilters) return res;
@@ -85,10 +85,6 @@ export default function validateSplitFilters(splitFilters, mode) {
   }
 
   // Validate filters and group their values by filter type inside `groupedFilters` object
-  FILTERS_METADATA.forEach(function (metadata) {
-    res.groupedFilters[metadata.type] = [];
-  });
-  // Assign the valid filters to the output of the validator by using filter function
   res.validFilters = splitFilters.filter((filter, index) => {
     if (filter && validateFilterType(filter.type) && Array.isArray(filter.values)) {
       res.groupedFilters[filter.type] = res.groupedFilters[filter.type].concat(filter.values);

package/src/utils/logger/LoggerFactory.js

@@ -5,17 +5,28 @@ import objectAssign from 'object-assign';
 
 export const LogLevels = {
   'DEBUG': 'DEBUG',
-  'INFO':  'INFO',
-  'WARN':  'WARN',
+  'INFO': 'INFO',
+  'WARN': 'WARN',
   'ERROR': 'ERROR',
   'NONE': 'NONE'
 };
 
+const LogLevelIndexes = {
+  DEBUG: 1,
+  INFO: 2,
+  WARN: 3,
+  ERROR: 4,
+  NONE: 5
+};
+
 // DEBUG is the default. The log level is not specific to an SDK instance.
-let GlobalLogLevel = LogLevels.DEBUG;
+let GlobalLogLevel = LogLevelIndexes.DEBUG;
 
+/**
+ * @param {'DEBUG'|'INFO'|'WARN'|'ERROR'|'NONE'} level
+ */
 export const setLogLevel = (level) => {
-  GlobalLogLevel = level;
+  GlobalLogLevel = LogLevelIndexes[level];
 };
 
 const defaultOptions = {
@@ -30,22 +41,22 @@ export class Logger {
   }
 
   debug(msg) {
-    if(this._shouldLog(LogLevels.DEBUG))
+    if (this._shouldLog(LogLevelIndexes.DEBUG))
       this._log(LogLevels.DEBUG, msg);
   }
 
   info(msg) {
-    if(this._shouldLog(LogLevels.INFO))
+    if (this._shouldLog(LogLevelIndexes.INFO))
       this._log(LogLevels.INFO, msg);
   }
 
   warn(msg) {
-    if(this._shouldLog(LogLevels.WARN))
+    if (this._shouldLog(LogLevelIndexes.WARN))
       this._log(LogLevels.WARN, msg);
   }
 
   error(msg) {
-    if(this.options.displayAllErrors || this._shouldLog(LogLevels.ERROR))
+    if (this.options.displayAllErrors || this._shouldLog(LogLevelIndexes.ERROR))
       this._log(LogLevels.ERROR, msg);
   }
 
@@ -59,8 +70,8 @@ export class Logger {
     const textPre = ' => ';
     let result = '';
 
-    if(this.options.showLevel) {
-      result += '[' + level +']' + (level === LogLevels.INFO || level === LogLevels.WARN ? ' ' : '') + ' ';
+    if (this.options.showLevel) {
+      result += '[' + level + ']' + (level === LogLevels.INFO || level === LogLevels.WARN ? ' ' : '') + ' ';
     }
 
     if (this.category) {
@@ -70,12 +81,10 @@ export class Logger {
     return result += text;
   }
 
+  /**
+   * @param {number} level
+   */
   _shouldLog(level) {
-    const logLevel = GlobalLogLevel;
-    const levels = Object.keys(LogLevels).map(f => LogLevels[f]);
-    const index = levels.indexOf(level); // What's the index of what it's trying to check if it should log
-    const levelIdx = levels.indexOf(logLevel); // What's the current log level index.
-
-    return index >= levelIdx;
+    return level >= GlobalLogLevel;
   }
 }

package/src/utils/settings/index.js

@@ -27,7 +27,7 @@ import { API } from '../../utils/logger';
 import { STANDALONE_MODE, STORAGE_MEMORY, CONSUMER_MODE, OPTIMIZED } from '../../utils/constants';
 import validImpressionsMode from './impressionsMode';
 
-const version = '10.15.10-rc.2';
+const version = '10.16.1-rc.0';
 const eventsEndpointMatcher = /^\/(testImpressions|metrics|events)/;
 const authEndpointMatcher = /^\/v2\/auth/;
 const streamingEndpointMatcher = /^\/(sse|event-stream)/;