@splitsoftware/splitio-commons 1.13.2-rc.1 → 1.13.2-rc.10

package/src/evaluator/matchers/semver_eq.ts

@@ -0,0 +1,13 @@
+import { Semver } from '../../utils/Semver';
+
+export function equalToSemverMatcherContext(ruleAttr: string) {
+  const ruleSemver = new Semver(ruleAttr);
+
+  return function equalToSemverMatcher(runtimeAttr: string): boolean {
+    const runtimeSemver = new Semver(runtimeAttr);
+
+    const isEqual = ruleSemver.version === runtimeSemver.version;
+
+    return isEqual;
+  };
+}

package/src/evaluator/matchers/semver_gte.ts

@@ -0,0 +1,13 @@
+import { Semver } from '../../utils/Semver';
+
+export function greaterThanEqualToSemverMatcherContext(ruleAttr: string) {
+  const ruleSemver = new Semver(ruleAttr);
+
+  return function greaterThanEqualToSemverMatcher(runtimeAttr: string): boolean {
+    const runtimeSemver = new Semver(runtimeAttr);
+
+    const isGreaterThanEqual = runtimeSemver.compare(ruleSemver) >= 0;
+
+    return isGreaterThanEqual;
+  };
+}

package/src/evaluator/matchers/semver_inlist.ts

@@ -0,0 +1,17 @@
+import { _Set } from '../../utils/lang/sets';
+import { Semver } from '../../utils/Semver';
+
+export function inListSemverMatcherContext(ruleAttr: string[]) {
+  // @TODO ruleAttr validation should be done at the `parser` or `matchersTransform` level to reuse for all matchers
+  if (!ruleAttr || ruleAttr.length === 0) throw new Error('whitelistMatcherData is required for IN_LIST_SEMVER matcher type');
+
+  const listOfSemvers = new _Set(ruleAttr.map((version) => new Semver(version).version));
+
+  return function inListSemverMatcher(runtimeAttr: string): boolean {
+    const runtimeSemver = new Semver(runtimeAttr).version;
+
+    const isInList = listOfSemvers.has(runtimeSemver);
+
+    return isInList;
+  };
+}

package/src/evaluator/matchers/semver_lte.ts

@@ -0,0 +1,13 @@
+import { Semver } from '../../utils/Semver';
+
+export function lessThanEqualToSemverMatcherContext(ruleAttr: string) {
+  const ruleSemver = new Semver(ruleAttr);
+
+  return function lessThanEqualToSemverMatcher(runtimeAttr: string): boolean {
+    const runtimeSemver = new Semver(runtimeAttr);
+
+    const isLessThanEqual = runtimeSemver.compare(ruleSemver) <= 0;
+
+    return isLessThanEqual;
+  };
+}

package/src/evaluator/matchers/string.ts

@@ -1,21 +1,8 @@
-import { ENGINE_MATCHER_STRING_INVALID, ENGINE_MATCHER_STRING } from '../../logger/constants';
-import { ILogger } from '../../logger/types';
+export function stringMatcherContext(ruleAttr: string) {
+  const regex = new RegExp(ruleAttr);
 
-export function stringMatcherContext(log: ILogger, ruleAttr: string) /*: Function */ {
   return function stringMatcher(runtimeAttr: string): boolean {
-    let re;
-
-    try {
-      re = new RegExp(ruleAttr);
-    } catch (e) {
-      log.debug(ENGINE_MATCHER_STRING_INVALID, [ruleAttr]);
-
-      return false;
-    }
-
-    let regexMatches = re.test(runtimeAttr);
-
-    log.debug(ENGINE_MATCHER_STRING, [runtimeAttr, ruleAttr, regexMatches ? 'yes' : 'no']);
+    const regexMatches = regex.test(runtimeAttr);
 
     return regexMatches;
   };

package/src/evaluator/matchers/sw.ts

@@ -1,12 +1,8 @@
-import { ENGINE_MATCHER_STARTS_WITH } from '../../logger/constants';
-import { ILogger } from '../../logger/types';
 import { startsWith } from '../../utils/lang';
 
-export function startsWithMatcherContext(log: ILogger, ruleAttr: string[]) /*: Function */ {
+export function startsWithMatcherContext(ruleAttr: string[]) {
   return function startsWithMatcher(runtimeAttr: string): boolean {
-    let matches = ruleAttr.some(e => startsWith(runtimeAttr, e));
-
-    log.debug(ENGINE_MATCHER_STARTS_WITH, [runtimeAttr, ruleAttr, matches]);
+    const matches = ruleAttr.some(e => startsWith(runtimeAttr, e));
 
     return matches;
   };

package/src/evaluator/matchers/whitelist.ts

@@ -1,12 +1,10 @@
-import { setToArray, ISet } from '../../utils/lang/sets';
-import { ILogger } from '../../logger/types';
-import { ENGINE_MATCHER_WHITELIST } from '../../logger/constants';
+import { _Set } from '../../utils/lang/sets';
 
-export function whitelistMatcherContext(log: ILogger, ruleAttr: ISet<string>) /*: Function */ {
-  return function whitelistMatcher(runtimeAttr: string): boolean {
-    let isInWhitelist = ruleAttr.has(runtimeAttr);
+export function whitelistMatcherContext(ruleAttr: string[]) {
+  const whitelistSet = new _Set(ruleAttr);
 
-    log.debug(ENGINE_MATCHER_WHITELIST, [runtimeAttr, setToArray(ruleAttr).join(','), isInWhitelist]);
+  return function whitelistMatcher(runtimeAttr: string): boolean {
+    const isInWhitelist = whitelistSet.has(runtimeAttr);
 
     return isInWhitelist;
   };

package/src/evaluator/matchersTransform/index.ts

@@ -2,10 +2,9 @@ import { findIndex } from '../../utils/lang';
 import { matcherTypes, matcherTypesMapper, matcherDataTypes } from '../matchers/matcherTypes';
 import { segmentTransform } from './segment';
 import { whitelistTransform } from './whitelist';
-import { setTransform } from './set';
 import { numericTransform } from './unaryNumeric';
 import { zeroSinceHH, zeroSinceSS } from '../convertions';
-import { IBetweenMatcherData, IInSegmentMatcherData, ISplitMatcher, IUnaryNumericMatcherData, IWhitelistMatcherData } from '../../dtos/types';
+import { IBetweenMatcherData, IInSegmentMatcherData, ISplitMatcher, IUnaryNumericMatcherData } from '../../dtos/types';
 import { IMatcherDto } from '../types';
 
 /**
@@ -15,16 +14,17 @@ export function matchersTransform(matchers: ISplitMatcher[]): IMatcherDto[] {
 
   let parsedMatchers = matchers.map(matcher => {
     let {
-      matcherType                                   /* string */,
-      negate                                        /* boolean */,
-      keySelector                                   /* keySelectorObject */,
-      userDefinedSegmentMatcherData: segmentObject  /* segmentObject */,
-      whitelistMatcherData: whitelistObject         /* whiteListObject, provided by 'WHITELIST', set and string matchers */,
-      unaryNumericMatcherData: unaryNumericObject   /* unaryNumericObject */,
-      betweenMatcherData: betweenObject             /* betweenObject */,
-      dependencyMatcherData: dependencyObject       /* dependencyObject */,
+      matcherType,
+      negate,
+      keySelector,
+      userDefinedSegmentMatcherData,
+      whitelistMatcherData, /* whitelistObject, provided by 'WHITELIST', 'IN_LIST_SEMVER', set and string matchers */
+      unaryNumericMatcherData,
+      betweenMatcherData,
+      dependencyMatcherData,
       booleanMatcherData,
-      stringMatcherData
+      stringMatcherData,
+      betweenStringMatcherData
     } = matcher;
 
     let attribute = keySelector && keySelector.attribute;
@@ -34,28 +34,26 @@ export function matchersTransform(matchers: ISplitMatcher[]): IMatcherDto[] {
     let value = undefined;
 
     if (type === matcherTypes.IN_SEGMENT) {
-      value = segmentTransform(segmentObject as IInSegmentMatcherData);
-    } else if (type === matcherTypes.WHITELIST) {
-      value = whitelistTransform(whitelistObject as IWhitelistMatcherData);
+      value = segmentTransform(userDefinedSegmentMatcherData as IInSegmentMatcherData);
     } else if (type === matcherTypes.EQUAL_TO) {
-      value = numericTransform(unaryNumericObject as IUnaryNumericMatcherData);
+      value = numericTransform(unaryNumericMatcherData as IUnaryNumericMatcherData);
       dataType = matcherDataTypes.NUMBER;
 
-      if ((unaryNumericObject as IUnaryNumericMatcherData).dataType === 'DATETIME') {
+      if ((unaryNumericMatcherData as IUnaryNumericMatcherData).dataType === 'DATETIME') {
         value = zeroSinceHH(value);
         dataType = matcherDataTypes.DATETIME;
       }
     } else if (type === matcherTypes.GREATER_THAN_OR_EQUAL_TO ||
       type === matcherTypes.LESS_THAN_OR_EQUAL_TO) {
-      value = numericTransform(unaryNumericObject as IUnaryNumericMatcherData);
+      value = numericTransform(unaryNumericMatcherData as IUnaryNumericMatcherData);
       dataType = matcherDataTypes.NUMBER;
 
-      if ((unaryNumericObject as IUnaryNumericMatcherData).dataType === 'DATETIME') {
+      if ((unaryNumericMatcherData as IUnaryNumericMatcherData).dataType === 'DATETIME') {
         value = zeroSinceSS(value);
         dataType = matcherDataTypes.DATETIME;
       }
     } else if (type === matcherTypes.BETWEEN) {
-      value = betweenObject as IBetweenMatcherData;
+      value = betweenMatcherData as IBetweenMatcherData;
       dataType = matcherDataTypes.NUMBER;
 
       if (value.dataType === 'DATETIME') {
@@ -63,27 +61,36 @@ export function matchersTransform(matchers: ISplitMatcher[]): IMatcherDto[] {
         value.end = zeroSinceSS(value.end);
         dataType = matcherDataTypes.DATETIME;
       }
+    } else if (type === matcherTypes.BETWEEN_SEMVER) {
+      value = betweenStringMatcherData;
     } else if (
       type === matcherTypes.EQUAL_TO_SET ||
       type === matcherTypes.CONTAINS_ANY_OF_SET ||
       type === matcherTypes.CONTAINS_ALL_OF_SET ||
       type === matcherTypes.PART_OF_SET
     ) {
-      value = setTransform(whitelistObject as IWhitelistMatcherData);
+      value = whitelistTransform(whitelistMatcherData);
       dataType = matcherDataTypes.SET;
     } else if (
+      type === matcherTypes.WHITELIST ||
+      type === matcherTypes.IN_LIST_SEMVER ||
       type === matcherTypes.STARTS_WITH ||
       type === matcherTypes.ENDS_WITH ||
       type === matcherTypes.CONTAINS_STRING
     ) {
-      value = setTransform(whitelistObject as IWhitelistMatcherData);
+      value = whitelistTransform(whitelistMatcherData);
     } else if (type === matcherTypes.IN_SPLIT_TREATMENT) {
-      value = dependencyObject;
+      value = dependencyMatcherData;
       dataType = matcherDataTypes.NOT_SPECIFIED;
     } else if (type === matcherTypes.EQUAL_TO_BOOLEAN) {
       dataType = matcherDataTypes.BOOLEAN;
       value = booleanMatcherData;
-    } else if (type === matcherTypes.MATCHES_STRING) {
+    } else if (
+      type === matcherTypes.MATCHES_STRING ||
+      type === matcherTypes.EQUAL_TO_SEMVER ||
+      type === matcherTypes.GREATER_THAN_OR_EQUAL_TO_SEMVER ||
+      type === matcherTypes.LESS_THAN_OR_EQUAL_TO_SEMVER
+    ) {
       value = stringMatcherData;
     }
 
@@ -91,6 +98,7 @@ export function matchersTransform(matchers: ISplitMatcher[]): IMatcherDto[] {
       attribute,        // attribute over we should do the matching, undefined means 'use the key'
       negate,           // should we negate the result?
       type,             // which kind of matcher we should evaluate
+      name: matcherType,// name of the matcher for logging purposes
       value,            // metadata used for the matching
       dataType          // runtime input data type
     };

package/src/evaluator/matchersTransform/whitelist.ts

@@ -1,9 +1,8 @@
-import { IWhitelistMatcherData } from '../../dtos/types';
-import { _Set } from '../../utils/lang/sets';
+import { ISplitMatcher } from '../../dtos/types';
 
 /**
- * Extract whitelist as a set. Used by 'WHITELIST' matcher.
+ * Extract whitelist array.
  */
-export function whitelistTransform(whitelistObject: IWhitelistMatcherData) {
-  return new _Set(whitelistObject.whitelist);
+export function whitelistTransform(whitelistObject: ISplitMatcher['whitelistMatcherData']) {
+  return whitelistObject && whitelistObject.whitelist;
 }

package/src/evaluator/parser/index.ts

@@ -7,10 +7,11 @@ import { ifElseIfCombinerContext } from '../combiners/ifelseif';
 import { andCombinerContext } from '../combiners/and';
 import { thenable } from '../../utils/promise/thenable';
 import { IEvaluator, IMatcherDto, ISplitEvaluator } from '../types';
-import { ISplitCondition } from '../../dtos/types';
+import { ISplitCondition, MaybeThenable } from '../../dtos/types';
 import { IStorageAsync, IStorageSync } from '../../storages/types';
 import { SplitIO } from '../../types';
 import { ILogger } from '../../logger/types';
+import { ENGINE_MATCHER_ERROR, ENGINE_MATCHER_RESULT } from '../../logger/constants';
 
 export function parser(log: ILogger, conditions: ISplitCondition[], storage: IStorageSync | IStorageAsync): IEvaluator {
   let predicates = [];
@@ -28,19 +29,34 @@ export function parser(log: ILogger, conditions: ISplitCondition[], storage: ISt
 
     // create a set of pure functions from the matcher's dto
     const expressions = matchers.map((matcherDto: IMatcherDto) => {
-      const matcher = matcherFactory(log, matcherDto, storage);
+      let matcher: ReturnType<typeof matcherFactory>;
+      try {
+        matcher = matcherFactory(log, matcherDto, storage);
+      } catch (error) {
+        log.error(ENGINE_MATCHER_ERROR, [matcherDto.name, error]);
+      }
 
       // Evaluator function.
       return (key: string, attributes: SplitIO.Attributes | undefined, splitEvaluator: ISplitEvaluator) => {
         const value = sanitizeValue(log, key, matcherDto, attributes);
-        const result = value !== undefined && matcher ? matcher(value, splitEvaluator) : false;
+        let result: MaybeThenable<boolean> = false;
 
-        if (thenable(result)) {
-          // @ts-ignore
-          return result.then(res => Boolean(res ^ matcherDto.negate));
+        if (value !== undefined && matcher) {
+          try {
+            result = matcher(value, splitEvaluator);
+          } catch (error) {
+            log.error(ENGINE_MATCHER_ERROR, [matcherDto.name, error]);
+          }
         }
-        // @ts-ignore
-        return Boolean(result ^ matcherDto.negate);
+
+        function handleResult(result: boolean) {
+          log.debug(ENGINE_MATCHER_RESULT, [matcherDto.name, result, matcherDto.value, value]); // @ts-ignore
+          return Boolean(result ^ matcherDto.negate);
+        }
+
+        return thenable(result) ?
+          result.then(handleResult) :
+          handleResult(result);
       };
     });
 

package/src/evaluator/types.ts

@@ -1,6 +1,5 @@
-import { IBetweenMatcherData, IDependencyMatcherData, MaybeThenable } from '../dtos/types';
+import { IBetweenMatcherData, IBetweenStringMatcherData, IDependencyMatcherData, MaybeThenable } from '../dtos/types';
 import { IStorageAsync, IStorageSync } from '../storages/types';
-import { ISet } from '../utils/lang/sets';
 import { SplitIO } from '../types';
 import { ILogger } from '../logger/types';
 
@@ -11,7 +10,8 @@ export interface IDependencyMatcherValue {
 
 export interface IMatcherDto {
   type: number
-  value?: string | number | boolean | string[] | IDependencyMatcherData | ISet<string> | IBetweenMatcherData | null
+  name: string
+  value?: string | number | boolean | string[] | IDependencyMatcherData | IBetweenMatcherData | IBetweenStringMatcherData | null
 
   attribute: string | null
   negate: boolean

package/src/logger/constants.ts

@@ -9,25 +9,8 @@ export const ENGINE_COMBINER_AND = 0;
 export const ENGINE_COMBINER_IFELSEIF = 1;
 export const ENGINE_COMBINER_IFELSEIF_NO_TREATMENT = 2;
 export const ENGINE_BUCKET = 3;
-export const ENGINE_MATCHER_ALL = 4;
-export const ENGINE_MATCHER_BETWEEN = 5;
-export const ENGINE_MATCHER_BOOLEAN = 6;
-export const ENGINE_MATCHER_CONTAINS_ALL = 7;
-export const ENGINE_MATCHER_CONTAINS_ANY = 8;
-export const ENGINE_MATCHER_CONTAINS_STRING = 9;
 export const ENGINE_MATCHER_DEPENDENCY = 10;
 export const ENGINE_MATCHER_DEPENDENCY_PRE = 11;
-export const ENGINE_MATCHER_EQUAL = 12;
-export const ENGINE_MATCHER_EQUAL_TO_SET = 13;
-export const ENGINE_MATCHER_ENDS_WITH = 14;
-export const ENGINE_MATCHER_GREATER = 15;
-export const ENGINE_MATCHER_LESS = 16;
-export const ENGINE_MATCHER_PART_OF = 17;
-export const ENGINE_MATCHER_SEGMENT = 18;
-export const ENGINE_MATCHER_STRING = 19;
-export const ENGINE_MATCHER_STRING_INVALID = 20;
-export const ENGINE_MATCHER_STARTS_WITH = 21;
-export const ENGINE_MATCHER_WHITELIST = 22;
 export const ENGINE_VALUE = 23;
 export const ENGINE_SANITIZE = 24;
 export const CLEANUP_REGISTERING = 25;
@@ -45,6 +28,7 @@ export const SYNC_TASK_START = 36;
 export const SYNC_TASK_EXECUTE = 37;
 export const SYNC_TASK_STOP = 38;
 export const SETTINGS_SPLITS_FILTER = 39;
+export const ENGINE_MATCHER_RESULT = 40;
 
 export const CLIENT_READY_FROM_CACHE = 100;
 export const CLIENT_READY = 101;
@@ -131,6 +115,7 @@ export const ERROR_NOT_BOOLEAN = 325;
 export const ERROR_MIN_CONFIG_PARAM = 326;
 export const ERROR_TOO_MANY_SETS = 327;
 export const ERROR_SETS_FILTER_EXCLUSIVE = 328;
+export const ENGINE_MATCHER_ERROR = 329;
 
 // Log prefixes (a.k.a. tags or categories)
 export const LOG_PREFIX_SETTINGS = 'settings';

package/src/logger/index.ts

@@ -1,6 +1,6 @@
 import { objectAssign } from '../utils/lang/objectAssign';
 import { ILoggerOptions, ILogger } from './types';
-import { find } from '../utils/lang';
+import { find, isObject } from '../utils/lang';
 import { LogLevel } from '../types';
 import { IMap, _Map } from '../utils/lang/maps';
 
@@ -28,7 +28,13 @@ export function isLogLevelString(str: string): str is LogLevel {
 export function _sprintf(format: string = '', args: any[] = []): string {
   let i = 0;
   return format.replace(/%s/g, function () {
-    return args[i++];
+    let arg = args[i++];
+    if (isObject(arg) || Array.isArray(arg)) {
+      try {
+        arg = JSON.stringify(arg);
+      } catch (e) { /* empty */ }
+    }
+    return arg;
   });
 }
 

package/src/logger/messages/debug.ts

@@ -7,27 +7,11 @@ export const codesDebug: [number, string][] = codesInfo.concat([
   [c.ENGINE_COMBINER_IFELSEIF, c.LOG_PREFIX_ENGINE_COMBINER + 'Treatment found: %s'],
   [c.ENGINE_COMBINER_IFELSEIF_NO_TREATMENT, c.LOG_PREFIX_ENGINE_COMBINER + 'All predicates evaluated, no treatment found.'],
   [c.ENGINE_BUCKET, c.LOG_PREFIX_ENGINE + ': using algo "murmur" bucket %s for key %s using seed %s - treatment %s'],
-  [c.ENGINE_MATCHER_ALL, c.LOG_PREFIX_ENGINE_MATCHER + '[allMatcher] is always true'],
-  [c.ENGINE_MATCHER_BETWEEN, c.LOG_PREFIX_ENGINE_MATCHER + '[betweenMatcher] is %s between %s and %s? %s'],
-  [c.ENGINE_MATCHER_BOOLEAN, c.LOG_PREFIX_ENGINE_MATCHER + '[booleanMatcher] %s === %s'],
-  [c.ENGINE_MATCHER_CONTAINS_ALL, c.LOG_PREFIX_ENGINE_MATCHER + '[containsAllMatcher] %s contains all elements of %s? %s'],
-  [c.ENGINE_MATCHER_CONTAINS_ANY, c.LOG_PREFIX_ENGINE_MATCHER + '[containsAnyMatcher] %s contains at least an element of %s? %s'],
-  [c.ENGINE_MATCHER_CONTAINS_STRING, c.LOG_PREFIX_ENGINE_MATCHER + '[containsStringMatcher] %s contains %s? %s'],
-  [c.ENGINE_MATCHER_DEPENDENCY, c.LOG_PREFIX_ENGINE_MATCHER + '[dependencyMatcher] parent feature flag "%s" evaluated to "%s" with label "%s". %s evaluated treatment is part of [%s] ? %s.'],
-  [c.ENGINE_MATCHER_DEPENDENCY_PRE, c.LOG_PREFIX_ENGINE_MATCHER + '[dependencyMatcher] will evaluate parent feature flag: "%s" with key: %s %s'],
-  [c.ENGINE_MATCHER_EQUAL, c.LOG_PREFIX_ENGINE_MATCHER + '[equalToMatcher] is %s equal to %s? %s'],
-  [c.ENGINE_MATCHER_EQUAL_TO_SET, c.LOG_PREFIX_ENGINE_MATCHER + '[equalToSetMatcher] is %s equal to set %s? %s'],
-  [c.ENGINE_MATCHER_ENDS_WITH, c.LOG_PREFIX_ENGINE_MATCHER + '[endsWithMatcher] %s ends with %s? %s'],
-  [c.ENGINE_MATCHER_GREATER, c.LOG_PREFIX_ENGINE_MATCHER + '[greaterThanEqualMatcher] is %s greater than %s? %s'],
-  [c.ENGINE_MATCHER_LESS, c.LOG_PREFIX_ENGINE_MATCHER + '[lessThanEqualMatcher] is %s less than %s? %s'],
-  [c.ENGINE_MATCHER_PART_OF, c.LOG_PREFIX_ENGINE_MATCHER + '[partOfMatcher] %s is part of %s? %s'],
-  [c.ENGINE_MATCHER_SEGMENT, c.LOG_PREFIX_ENGINE_MATCHER + '[segmentMatcher] evaluated %s / %s => %s'],
-  [c.ENGINE_MATCHER_STRING, c.LOG_PREFIX_ENGINE_MATCHER + '[stringMatcher] does %s matches with %s? %s'],
-  [c.ENGINE_MATCHER_STRING_INVALID, c.LOG_PREFIX_ENGINE_MATCHER + '[stringMatcher] %s is an invalid regex'],
-  [c.ENGINE_MATCHER_STARTS_WITH, c.LOG_PREFIX_ENGINE_MATCHER + '[startsWithMatcher] %s starts with %s? %s'],
-  [c.ENGINE_MATCHER_WHITELIST, c.LOG_PREFIX_ENGINE_MATCHER + '[whitelistMatcher] evaluated %s in [%s] => %s'],
-  [c.ENGINE_VALUE, c.LOG_PREFIX_ENGINE_VALUE + 'Extracted attribute [%s], [%s] will be used for matching.'],
-  [c.ENGINE_SANITIZE, c.LOG_PREFIX_ENGINE + ':sanitize: Attempted to sanitize [%s] which should be of type [%s]. Sanitized and processed value => [%s]'],
+  [c.ENGINE_MATCHER_DEPENDENCY, c.LOG_PREFIX_ENGINE_MATCHER + '[IN_SPLIT_TREATMENT] parent feature flag "%s" evaluated to "%s" with label "%s". %s evaluated treatment is part of %s ? %s.'],
+  [c.ENGINE_MATCHER_DEPENDENCY_PRE, c.LOG_PREFIX_ENGINE_MATCHER + '[IN_SPLIT_TREATMENT] will evaluate parent feature flag: "%s" with key: %s %s'],
+  [c.ENGINE_VALUE, c.LOG_PREFIX_ENGINE_VALUE + 'Extracted attribute `%s`. %s will be used for matching.'],
+  [c.ENGINE_SANITIZE, c.LOG_PREFIX_ENGINE + ':sanitize: Attempted to sanitize %s which should be of type %s. Sanitized and processed value => %s'],
+  [c.ENGINE_MATCHER_RESULT, c.LOG_PREFIX_ENGINE_MATCHER + '[%s] Result: %s. Rule value: %s. Evaluation value: %s'],
   // SDK
   [c.CLEANUP_REGISTERING, c.LOG_PREFIX_CLEANUP + 'Registering cleanup handler %s'],
   [c.CLEANUP_DEREGISTERING, c.LOG_PREFIX_CLEANUP + 'Deregistering cleanup handler %s'],

package/src/logger/messages/error.ts

@@ -2,7 +2,8 @@ import * as c from '../constants';
 
 export const codesError: [number, string][] = [
   // evaluator
-  [c.ERROR_ENGINE_COMBINER_IFELSEIF, c.LOG_PREFIX_ENGINE_COMBINER + 'Invalid feature flag, no valid rules or unsupported matcher type found'],
+  [c.ERROR_ENGINE_COMBINER_IFELSEIF, c.LOG_PREFIX_ENGINE_COMBINER + 'Invalid feature flag, no valid rules or unsupported targeting rule type found'],
+  [c.ENGINE_MATCHER_ERROR, c.LOG_PREFIX_ENGINE_MATCHER + '[%s] %s'],
   // SDK
   [c.ERROR_LOGLEVEL_INVALID, 'logger: Invalid Log Level - No changes to the logs will be applied.'],
   [c.ERROR_CLIENT_CANNOT_GET_READY, 'The SDK will not get ready. Reason: %s'],

package/src/logger/messages/warn.ts

@@ -4,7 +4,7 @@ import { codesError } from './error';
 export const codesWarn: [number, string][] = codesError.concat([
   // evaluator
   [c.ENGINE_VALUE_INVALID, c.LOG_PREFIX_ENGINE_VALUE + 'Value %s doesn\'t match with expected type.'],
-  [c.ENGINE_VALUE_NO_ATTRIBUTES, c.LOG_PREFIX_ENGINE_VALUE + 'Defined attribute [%s], no attributes received.'],
+  [c.ENGINE_VALUE_NO_ATTRIBUTES, c.LOG_PREFIX_ENGINE_VALUE + 'Defined attribute `%s`. No attributes received.'],
   // synchronizer
   [c.SYNC_MYSEGMENTS_FETCH_RETRY, c.LOG_PREFIX_SYNC_MYSEGMENTS + 'Retrying download of segments #%s. Reason: %s'],
   [c.SYNC_SPLITS_FETCH_FAILS, c.LOG_PREFIX_SYNC_SPLITS + 'Error while doing fetch of feature flags. %s'],

package/src/services/splitApi.ts

@@ -44,17 +44,16 @@ export function splitApiFactory(
     },
 
     fetchAuth(userMatchingKeys?: string[]) {
-      let url = `${urls.auth}/v2/auth`;
-      if (userMatchingKeys) { // accounting the possibility that `userMatchingKeys` is undefined (server-side API)
+      let url = `${urls.auth}/v2/auth?s=${FLAGS_SPEC}`;
+      if (userMatchingKeys) { // `userMatchingKeys` is undefined in server-side
         const queryParams = userMatchingKeys.map(userKeyToQueryParam).join('&');
-        if (queryParams) // accounting the possibility that `userKeys` and thus `queryParams` are empty
-          url += '?' + queryParams;
+        if (queryParams) url += '&' + queryParams;
       }
       return splitHttpClient(url, undefined, telemetryTracker.trackHttp(TOKEN));
     },
 
     fetchSplitChanges(since: number, noCache?: boolean, till?: number) {
-      const url = `${urls.sdk}/splitChanges?s=${FLAGS_SPEC}&since=${since}${till ? '&till=' + till : ''}${filterQueryString || ''}`;
+      const url = `${urls.sdk}/splitChanges?s=${FLAGS_SPEC}&since=${since}${filterQueryString || ''}${till ? '&till=' + till : ''}`;
       return splitHttpClient(url, noCache ? noCacheHeaderOptions : undefined, telemetryTracker.trackHttp(SPLITS))
         .catch((err) => {
           if (err.statusCode === 414) settings.log.error(ERROR_TOO_MANY_SETS);

package/src/storages/inLocalStorage/SplitsCacheInLocal.ts

@@ -146,7 +146,7 @@ export class SplitsCacheInLocal extends AbstractSplitsCacheSync {
 
     // when using a new split query, we must update it at the store
     if (this.updateNewFilter) {
-      this.log.info(LOG_PREFIX + 'SDK key or feature flag filter criteria was modified. Updating cache');
+      this.log.info(LOG_PREFIX + 'SDK key, flags filter criteria or flags spec version was modified. Updating cache');
       const storageHashKey = this.keys.buildHashKey();
       try {
         localStorage.setItem(storageHashKey, this.storageHash);

package/src/storages/pluggable/index.ts

@@ -96,7 +96,7 @@ export function PluggableStorage(options: PluggableStorageOptions): IStorageAsyn
         return wrapper.get(keys.buildHashKey()).then((hash) => {
           const currentHash = getStorageHash(settings);
           if (hash !== currentHash) {
-            log.info(LOG_PREFIX + 'Storage HASH has changed (SDK key or feature flag filter criteria was modified). Clearing cache');
+            log.info(LOG_PREFIX + 'Storage HASH has changed (SDK key, flags filter criteria or flags spec version was modified). Clearing cache');
             return wrapper.getKeysByPrefix(`${keys.prefix}.`).then(storageKeys => {
               return Promise.all(storageKeys.map(storageKey => wrapper.del(storageKey)));
             }).then(() => wrapper.set(keys.buildHashKey(), currentHash));

package/src/utils/Semver.ts

@@ -0,0 +1,111 @@
+import { isString } from '../utils/lang';
+
+const NUMERIC_IDENTIFIER_REGEX = /^[0-9]+$/;
+
+const METADATA_DELIMITER = '+';
+const PRERELEASE_DELIMITER = '-';
+const VALUE_DELIMITER = '.';
+
+/**
+ * Compares two strings. If both strings are numeric identifiers, they are compared numerically. Otherwise, they are compared lexicographically.
+ * This could be implemented using `a.localeCompare(b, undefined, { numeric: true })` but locale options are not broadly supported.
+ */
+function compareStrings(a: string, b: string): number {
+  if (NUMERIC_IDENTIFIER_REGEX.test(a) && NUMERIC_IDENTIFIER_REGEX.test(b)) {
+    const result = a.length - b.length;
+    if (result !== 0) {
+      return result;
+    }
+  }
+  return a < b ? -1 : a > b ? 1 : 0;
+}
+
+// Sanitizes a numeric identifier by removing leading zeros
+function sanitizeNumericIdentifier(value: string): string {
+  return value.replace(/^0+(?=\d)/, '');
+}
+
+function throwError(version: string) {
+  throw new Error('Unable to convert to Semver, incorrect format: ' + version);
+}
+
+export class Semver {
+
+  private readonly _major: string;
+  private readonly _minor: string;
+  private readonly _patch: string;
+  private readonly _preRelease: string[];
+  private readonly _isStable: boolean;
+
+  // Version string for 'equal' and 'in list' comparisons
+  public readonly version: string;
+
+  public constructor(version: string) {
+    if (!isString(version)) throwError(version);
+
+    // Separate metadata if exists
+    let index = version.indexOf(METADATA_DELIMITER);
+    let [vWithoutMetadata, metadata] = index === -1 ? [version] : [version.slice(0, index), version.slice(index + 1)];
+    if (metadata === '') throwError(version);
+
+    // Set pre-release versions if exists
+    index = vWithoutMetadata.indexOf(PRERELEASE_DELIMITER);
+    if (index === -1) {
+      this._isStable = true;
+      this._preRelease = [];
+    } else {
+      this._isStable = false;
+      this._preRelease = vWithoutMetadata.slice(index + 1).split(VALUE_DELIMITER).map((value) => {
+        if (!value) throwError(version);
+        return NUMERIC_IDENTIFIER_REGEX.test(value) ?
+          sanitizeNumericIdentifier(value) :
+          value;
+      });
+      vWithoutMetadata = vWithoutMetadata.slice(0, index);
+    }
+
+    // Set major, minor, and patch versions
+    const vParts = vWithoutMetadata.split(VALUE_DELIMITER).map((value) => {
+      if (!value || !NUMERIC_IDENTIFIER_REGEX.test(value)) throwError(version);
+      return sanitizeNumericIdentifier(value);
+    });
+
+    if (vParts.length !== 3) throwError(version);
+    this._major = vParts[0];
+    this._minor = vParts[1];
+    this._patch = vParts[2];
+
+    // Set version string
+    this.version = vParts.join(VALUE_DELIMITER);
+    if (this._preRelease.length) this.version += PRERELEASE_DELIMITER + this._preRelease.join(VALUE_DELIMITER);
+    if (metadata) this.version += METADATA_DELIMITER + metadata;
+  }
+
+  /**
+   * Precedence comparision between 2 Semver objects.
+   *
+   * @return `0` if `this === toCompare`, `-1` if `this < toCompare`, and `1` if `this > toCompare`
+   */
+  public compare(toCompare: Semver): number {
+    if (this.version === toCompare.version) return 0;
+
+    let result = compareStrings(this._major, toCompare._major);
+    if (result !== 0) return result;
+
+    result = compareStrings(this._minor, toCompare._minor);
+    if (result !== 0) return result;
+
+    result = compareStrings(this._patch, toCompare._patch);
+    if (result !== 0) return result;
+
+    if (!this._isStable && toCompare._isStable) return -1;
+    if (this._isStable && !toCompare._isStable) return 1;
+
+    for (let i = 0, length = Math.min(this._preRelease.length, toCompare._preRelease.length); i < length; i++) {
+      const result = compareStrings(this._preRelease[i], toCompare._preRelease[i]);
+      if (result !== 0) return result;
+    }
+
+    return this._preRelease.length - toCompare._preRelease.length;
+  }
+}

package/src/utils/labels/index.ts

@@ -5,4 +5,4 @@ export const SDK_NOT_READY = 'not ready';
 export const EXCEPTION = 'exception';
 export const SPLIT_ARCHIVED = 'archived';
 export const NOT_IN_SPLIT = 'not in split';
-export const UNSUPPORTED_MATCHER_TYPE = 'unsupported matcher type';
+export const UNSUPPORTED_MATCHER_TYPE = 'targeting rule type unsupported by sdk';

package/src/utils/settingsValidation/logger/pluggableLogger.ts

@@ -4,7 +4,7 @@ import { LogLevel } from '../../../types';
 import { getLogLevel } from './commons';
 
 function isLogger(log: any): log is ILogger {
-  return log && typeof log.debug === 'function' && typeof log.info === 'function' && typeof log.warn === 'function' && typeof log.error === 'function' && typeof log.setLogLevel === 'function';
+  return log !== null && typeof log === 'object' && typeof log.debug === 'function' && typeof log.info === 'function' && typeof log.warn === 'function' && typeof log.error === 'function' && typeof log.setLogLevel === 'function';
 }
 
 // By default it starts disabled.