@splitsoftware/splitio 10.17.4-rc.1 → 10.17.4-rc.4

package/CHANGES.txt

@@ -1,3 +1,20 @@
+10.18.0 (April 4, 2022)
+ - Updated internal dependencies to use @splitsoftware/splitio-commons dependency, which includes:
+      - Added support for user consent in client-side
+      - Added impressionsQueueSize config parameter
+      - Update SplitToGa integration ...
+      - Update modules to simplify the usage with bundling tools ...
+      - Updated dependencies for vulnerability fixes
+      - Bugfixing - Logger settings is configurable per factory instance instead of globally.
+      - Bugfixing - Wrongly logging warning 'No listeners for SDK Readiness detected' in consumer mode.
+      - Bugfixing - Updated internal `isObject` util that was returning false negatives on some environment, like Nuxt dev server or Miniflare.
+  - Added TLS support for Redis
+  -
+
+10.17.3 (February 25, 2022)
+ - Updated format for MySegments keys in LocalStorage, keeping backwards compatibility (issue https://github.com/splitio/javascript-client/issues/638).
+ - Updated some dependencies for vulnerability fixes.
+
 10.17.2 (January 31, 2022)
  - Updated some dependencies for vulnerability fixes.
  - Bugfixing - Fixed internal handling of Fetch API promise rejection, to avoid unhandled exceptions on some Web browsers (issue https://github.com/splitio/javascript-client/issues/621)

package/es/factory/browser.js

@@ -12,7 +12,7 @@ import { __InLocalStorageMockFactory } from '@splitsoftware/splitio-commons/esm/
 import { sdkFactory } from '@splitsoftware/splitio-commons/esm/sdkFactory';
 import { LOCALHOST_MODE, STORAGE_LOCALSTORAGE } from '@splitsoftware/splitio-commons/esm/utils/constants';
 import { shouldAddPt } from '@splitsoftware/splitio-commons/esm/trackers/impressionObserver/utils';
-import { userConsentProps } from '@splitsoftware/splitio-commons/esm/sdkFactory/userConsentProps';
+import { createUserConsentAPI } from '@splitsoftware/splitio-commons/esm/consent/sdkUserConsent';
 import { settingsFactory } from '../settings/browser';
 import { platform, SignalListener } from '../platform';
 var syncManagerOnlineCSFactory = syncManagerOnlineFactory(pollingManagerCSFactory, pushManagerFactory);
@@ -39,7 +39,11 @@ function getModules(settings) {
         SignalListener: SignalListener,
         integrationsManagerFactory: settings.integrations && settings.integrations.length > 0 ? integrationsManagerFactory.bind(null, settings.integrations) : undefined,
         impressionsObserverFactory: shouldAddPt(settings) ? impressionObserverCSFactory : undefined,
-        extraProps: userConsentProps
+        extraProps: function (params) {
+            return {
+                UserConsent: createUserConsentAPI(params)
+            };
+        }
     };
     switch (settings.mode) {
         case LOCALHOST_MODE:

package/es/settings/browser.js

@@ -8,6 +8,7 @@ import { LocalhostFromObject } from '@splitsoftware/splitio-commons/esm/sync/off
 import { validateConsent } from '@splitsoftware/splitio-commons/esm/utils/settingsValidation/consent';
 var params = {
     defaults: defaults,
+    acceptKey: true, acceptTT: true,
     runtime: validateRuntime,
     storage: validateStorage,
     integrations: validateIntegrations,

package/es/settings/defaults/version.js

@@ -1 +1 @@
-export var packageVersion = '10.17.4-rc.1';
+export var packageVersion = '10.17.4-rc.4';

package/es/settings/storage/node.js

@@ -12,7 +12,7 @@ export function validateStorage(settings) {
                     prefix: prefix
                 };
             }
-            var host = options.host, port = options.port, db = options.db, pass = options.pass, url = options.url, connectionTimeout = options.connectionTimeout, operationTimeout = options.operationTimeout;
+            var host = options.host, port = options.port, db = options.db, pass = options.pass, url = options.url, tls = options.tls, connectionTimeout = options.connectionTimeout, operationTimeout = options.operationTimeout;
             if (process.env.REDIS_HOST)
                 host = process.env.REDIS_HOST;
             if (process.env.REDIS_PORT)
@@ -36,6 +36,9 @@ export function validateStorage(settings) {
                 newOpts.db = db;
                 newOpts.pass = pass;
             }
+            if (tls) {
+                newOpts.tls = tls;
+            }
             return {
                 type: type,
                 prefix: prefix,

package/lib/factory/browser.js

@@ -15,7 +15,7 @@ var storageCS_1 = require("@splitsoftware/splitio-commons/cjs/utils/settingsVali
 var sdkFactory_1 = require("@splitsoftware/splitio-commons/cjs/sdkFactory");
 var constants_1 = require("@splitsoftware/splitio-commons/cjs/utils/constants");
 var utils_1 = require("@splitsoftware/splitio-commons/cjs/trackers/impressionObserver/utils");
-var userConsentProps_1 = require("@splitsoftware/splitio-commons/cjs/sdkFactory/userConsentProps");
+var sdkUserConsent_1 = require("@splitsoftware/splitio-commons/cjs/consent/sdkUserConsent");
 var browser_2 = require("../settings/browser");
 var platform_1 = require("../platform");
 var syncManagerOnlineCSFactory = (0, syncManagerOnline_1.syncManagerOnlineFactory)(pollingManagerCS_1.pollingManagerCSFactory, pushManager_1.pushManagerFactory);
@@ -42,7 +42,11 @@ function getModules(settings) {
         SignalListener: platform_1.SignalListener,
         integrationsManagerFactory: settings.integrations && settings.integrations.length > 0 ? browser_1.integrationsManagerFactory.bind(null, settings.integrations) : undefined,
         impressionsObserverFactory: (0, utils_1.shouldAddPt)(settings) ? impressionObserverCS_1.impressionObserverCSFactory : undefined,
-        extraProps: userConsentProps_1.userConsentProps
+        extraProps: function (params) {
+            return {
+                UserConsent: (0, sdkUserConsent_1.createUserConsentAPI)(params)
+            };
+        }
     };
     switch (settings.mode) {
         case constants_1.LOCALHOST_MODE:

package/lib/settings/browser.js

@@ -11,6 +11,7 @@ var LocalhostFromObject_1 = require("@splitsoftware/splitio-commons/cjs/sync/off
 var consent_1 = require("@splitsoftware/splitio-commons/cjs/utils/settingsValidation/consent");
 var params = {
     defaults: browser_1.defaults,
+    acceptKey: true, acceptTT: true,
     runtime: runtime_1.validateRuntime,
     storage: browser_2.validateStorage,
     integrations: browser_3.validateIntegrations,

package/lib/settings/defaults/version.js

@@ -1,4 +1,4 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.packageVersion = void 0;
-exports.packageVersion = '10.17.4-rc.1';
+exports.packageVersion = '10.17.4-rc.4';

package/lib/settings/storage/node.js

@@ -15,7 +15,7 @@ function validateStorage(settings) {
                     prefix: prefix
                 };
             }
-            var host = options.host, port = options.port, db = options.db, pass = options.pass, url = options.url, connectionTimeout = options.connectionTimeout, operationTimeout = options.operationTimeout;
+            var host = options.host, port = options.port, db = options.db, pass = options.pass, url = options.url, tls = options.tls, connectionTimeout = options.connectionTimeout, operationTimeout = options.operationTimeout;
             if (process.env.REDIS_HOST)
                 host = process.env.REDIS_HOST;
             if (process.env.REDIS_PORT)
@@ -39,6 +39,9 @@ function validateStorage(settings) {
                 newOpts.db = db;
                 newOpts.pass = pass;
             }
+            if (tls) {
+                newOpts.tls = tls;
+            }
             return {
                 type: type,
                 prefix: prefix,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@splitsoftware/splitio",
-  "version": "10.17.4-rc.1",
+  "version": "10.17.4-rc.4",
   "description": "Split SDK",
   "files": [
     "README.md",
@@ -10,9 +10,7 @@
     "lib",
     "types",
     "es",
-    "src",
-    "client",
-    "server"
+    "src"
   ],
   "repository": "splitio/javascript-client",
   "homepage": "https://github.com/splitio/javascript-client#readme",
@@ -34,8 +32,9 @@
     "node": ">=6"
   },
   "dependencies": {
-    "@splitsoftware/splitio-commons": "1.2.1-rc.8",
+    "@splitsoftware/splitio-commons": "1.2.1-rc.12",
     "@types/google.analytics": "0.0.40",
+    "@types/ioredis": "^4.28.0",
     "ioredis": "^4.28.0",
     "ip": "1.1.5",
     "js-yaml": "3.13.1",
@@ -46,7 +45,6 @@
     "eventsource": "^1.0.7"
   },
   "devDependencies": {
-    "@types/ioredis": "^4.28.0",
     "@types/node-fetch": "^2.5.12",
     "copyfiles": "^2.4.1",
     "cross-env": "^7.0.3",

package/src/factory/browser.js

@@ -12,7 +12,7 @@ import { __InLocalStorageMockFactory } from '@splitsoftware/splitio-commons/src/
 import { sdkFactory } from '@splitsoftware/splitio-commons/src/sdkFactory';
 import { LOCALHOST_MODE, STORAGE_LOCALSTORAGE } from '@splitsoftware/splitio-commons/src/utils/constants';
 import { shouldAddPt } from '@splitsoftware/splitio-commons/src/trackers/impressionObserver/utils';
-import { userConsentProps } from '@splitsoftware/splitio-commons/src/sdkFactory/userConsentProps';
+import { createUserConsentAPI } from '@splitsoftware/splitio-commons/src/consent/sdkUserConsent';
 
 import { settingsFactory } from '../settings/browser';
 import { platform, SignalListener } from '../platform';
@@ -54,7 +54,11 @@ function getModules(settings) {
 
     impressionsObserverFactory: shouldAddPt(settings) ? impressionObserverCSFactory : undefined,
 
-    extraProps: userConsentProps
+    extraProps: (params) => {
+      return {
+        UserConsent: createUserConsentAPI(params)
+      };
+    }
   };
 
   switch (settings.mode) {

package/src/settings/browser.js

@@ -9,6 +9,7 @@ import { validateConsent } from '@splitsoftware/splitio-commons/src/utils/settin
 
 const params = {
   defaults,
+  acceptKey: true, acceptTT: true, // Client with bound key and optional TT
   runtime: validateRuntime,
   storage: validateStorage,
   integrations: validateIntegrations,

package/src/settings/defaults/version.js

@@ -1 +1 @@
-export const packageVersion = '10.17.4-rc.1';
+export const packageVersion = '10.17.4-rc.4';

package/src/settings/storage/node.js

@@ -28,6 +28,7 @@ export function validateStorage(settings) {
         db,
         pass,
         url,
+        tls,
         connectionTimeout,
         operationTimeout
       } = options;
@@ -56,6 +57,10 @@ export function validateStorage(settings) {
         newOpts.pass = pass;
       }
 
+      if (tls) {
+        newOpts.tls = tls;
+      }
+
       return {
         type,
         prefix,

package/types/splitio.d.ts

@@ -1,8 +1,9 @@
-// Type definitions for Javascript and Node Split Software SDK
+// Type definitions for Javascript and NodeJS Split Software SDK
 // Project: http://www.split.io/
 // Definitions by: Nico Zelaya <https://github.com/NicoZelaya/>
 
 /// <reference types="google.analytics" />
+import { RedisOptions } from "ioredis";
 
 export as namespace SplitIO;
 export = SplitIO;
@@ -97,7 +98,7 @@ interface ISettings {
   readonly debug: boolean | LogLevel,
   readonly version: string,
   /**
-   * Mocked features map if using in browser, or mocked features file path string if using in Node.
+   * Mocked features map if using in browser, or mocked features file path string if using in NodeJS.
    */
   features: SplitIO.MockedFeaturesMap | SplitIO.MockedFeaturesFilePath,
   readonly streamingEnabled: boolean,
@@ -106,7 +107,7 @@ interface ISettings {
     impressionsMode: SplitIO.ImpressionsMode,
   }
   /**
-   * User consent status if using in browser. Undefined if using in Node.
+   * User consent status if using in browser. Undefined if using in NodeJS.
    */
   readonly userConsent?: SplitIO.ConsentStatus
 }
@@ -142,7 +143,38 @@ interface ILoggerAPI {
    * Log level constants. Use this to pass them to setLogLevel function.
    */
   LogLevel: {
-    [level: string]: LogLevel
+    [level in LogLevel]: LogLevel
+  }
+}
+/**
+ * User consent API
+ * @interface IUserConsentAPI
+ */
+interface IUserConsentAPI {
+  /**
+   * Set or update the user consent status. Possible values are `true` and `false`, which represent user consent `'GRANTED'` and `'DECLINED'` respectively.
+   * - `true ('GRANTED')`: the user has granted consent for tracking events and impressions. The SDK will send them to Split cloud.
+   * - `false ('DECLINED')`: the user has declined consent for tracking events and impressions. The SDK will not send them to Split cloud.
+   *
+   * NOTE: calling this method updates the user consent at a factory level, affecting all clients of the same factory.
+   *
+   * @function setStatus
+   * @param {boolean} userConsent The user consent status, true for 'GRANTED' and false for 'DECLINED'.
+   * @returns {boolean} Whether the provided param is a valid value (i.e., a boolean value) or not.
+   */
+  setStatus(userConsent: boolean): boolean;
+  /**
+   * Get the user consent status.
+   *
+   * @function getStatus
+   * @returns {ConsentStatus} The user consent status.
+   */
+  getStatus(): SplitIO.ConsentStatus;
+  /**
+   * Consent status constants. Use this to compare with the getStatus function result.
+   */
+  Status: {
+    [status in SplitIO.ConsentStatus]: SplitIO.ConsentStatus
   }
 }
 /**
@@ -325,7 +357,7 @@ interface INodeBasicSettings extends ISharedSettings {
     IPAddressesEnabled?: boolean
   },
   /**
-   * Defines which kind of storage we should instanciate.
+   * Defines which kind of storage we should instantiate.
    * @property {Object} storage
    */
   storage?: {
@@ -336,7 +368,7 @@ interface INodeBasicSettings extends ISharedSettings {
      */
     type?: StorageType,
     /**
-     * Options to be passed to the selected storage. Use it with type: 'REDIS'
+     * Options to be passed to the selected storage.
      * @property {Object} options
      */
     options?: Object,
@@ -985,7 +1017,8 @@ declare namespace SplitIO {
      */
     features?: MockedFeaturesMap,
     /**
-     * Defines which kind of storage we should instanciate.
+     * Defines which kind of storage we can instantiate on the browser.
+     * Possible storage types are 'MEMORY', which is the default, and 'LOCALSTORAGE'.
      * @property {Object} storage
      */
     storage?: {
@@ -1040,7 +1073,8 @@ declare namespace SplitIO {
      */
     urls?: UrlSettings,
     /**
-     * Defines which kind of storage we should instanciate.
+     * Defines which kind of storage we can instantiate on NodeJS for 'standalone' mode.
+     * The only possible storage type is 'MEMORY', which is the default.
      * @property {Object} storage
      */
     storage?: {
@@ -1074,6 +1108,11 @@ declare namespace SplitIO {
    * @see {@link https://help.split.io/hc/en-us/articles/360020564931-Node-js-SDK#configuration}
    */
   interface INodeAsyncSettings extends INodeBasicSettings {
+    /**
+     * Defines which kind of async storage we can instantiate on NodeJS for 'consumer' mode.
+     * The only possible storage type is 'REDIS'.
+     * @property {Object} storage
+     */
     storage: {
       /**
        * 'REDIS' storage type to be instantiated by the SDK.
@@ -1081,10 +1120,68 @@ declare namespace SplitIO {
        */
       type: NodeAsyncStorage,
       /**
-       * Options to be passed to the selected storage. Use it with type: 'REDIS'
+       * Options to be passed to the Redis storage. Use it with storage type: 'REDIS'.
        * @property {Object} options
        */
-      options?: Object,
+      options?: {
+        /**
+         * Redis URL. If set, `host`, `port`, `db` and `pass` params will be ignored.
+         *
+         * Examples:
+         * ```
+         *   url: 'localhost'
+         *   url: '127.0.0.1:6379'
+         *   url: 'redis://:authpassword@127.0.0.1:6379/0'
+         * ```
+         * @property {string=} url
+         */
+        url?: string,
+        /**
+         * Redis host.
+         * @property {string=} host
+         * @default 'localhost'
+         */
+        host?: string,
+        /**
+         * Redis port.
+         * @property {number=} port
+         * @default 6379
+         */
+        port?: number,
+        /**
+         * Redis database to be used.
+         * @property {number=} db
+         * @default 0
+         */
+        db?: number,
+        /**
+         * Redis password. Don't define if no password is used.
+         * @property {string=} pass
+         * @default undefined
+         */
+        pass?: string,
+        /**
+         * The milliseconds before a timeout occurs during the initial connection to the Redis server.
+         * @property {number=} connectionTimeout
+         * @default 10000
+         */
+        connectionTimeout?: number,
+        /**
+         * The milliseconds before Redis commands are timeout by the SDK.
+         * Method calls that involve Redis commands, like `client.getTreatment` or `client.track` calls, are resolved when the commands success or timeout.
+         * @property {number=} operationTimeout
+         * @default 5000
+         */
+        operationTimeout?: number,
+        /**
+         * TLS configuration for Redis connection.
+         * @see {@link https://www.npmjs.com/package/ioredis#tls-options }
+         *
+         * @property {Object=} tls
+         * @default undefined
+         */
+        tls?: RedisOptions['tls'],
+      },
       /**
        * Optional prefix to prevent any kind of data collision between SDK versions.
        * @property {string} prefix
@@ -1149,24 +1246,10 @@ declare namespace SplitIO {
      */
     client(key: SplitKey, trafficType?: string): IBrowserClient
     /**
-     * Set or update the user consent status. Possible values are `true` and `false`, which represent user consent `'GRANTED'` and `'DECLINED'` respectively.
-     * - `true ('GRANTED')`: the user has granted consent for tracking events and impressions. The SDK will send them to Split cloud.
-     * - `false ('DECLINED')`: the user has declined consent for tracking events and impressions. The SDK will not send them to Split cloud.
-     *
-     * NOTE: calling this method updates the user consent at a factory level, affecting all clients of the same factory.
-     *
-     * @function setUserConsent
-     * @param {boolean} userConsent The user consent status, true for 'GRANTED' and false for 'DECLINED'.
-     * @returns {boolean} Whether the provided param is a valid value (i.e., a boolean value) or not.
-     */
-    setUserConsent(userConsent: boolean): boolean;
-    /**
-     * Get the user consent status.
-     *
-     * @function getUserConsent
-     * @returns {ConsentStatus} The user consent status.
+     * User consent API.
+     * @property UserConsent
      */
-    getUserConsent(): ConsentStatus;
+    UserConsent: IUserConsentAPI
   }
   /**
    * This represents the interface for the SDK instance with asynchronous storage.

package/client/package.json

@@ -1,5 +0,0 @@
-{
-  "main": "../lib/factory/browser.js",
-  "module": "../es/factory/browser.js",
-  "types": "../types/client/index.d.ts"
-}

package/server/package.json

@@ -1,5 +0,0 @@
-{
-  "main": "../lib/factory/node.js",
-  "module": "../es/factory/node.js",
-  "types": "../types/server/index.d.ts"
-}

package/types/client/index.d.ts

@@ -1,14 +0,0 @@
-// Declaration file for Javascript Split Software SDK
-// Project: http://www.split.io/
-
-/// <reference path="../splitio.d.ts" />
-export = JsSdk;
-
-declare module JsSdk {
-  /**
-   * Split.io sdk factory function.
-   * The settings parameter should be an object that complies with the SplitIO.IBrowserSettings.
-   * For more information read the corresponding article: @see {@link https://help.split.io/hc/en-us/articles/360020448791-JavaScript-SDK#configuration}
-   */
-  export function SplitFactory(settings: SplitIO.IBrowserSettings): SplitIO.IBrowserSDK;
-}

package/types/server/index.d.ts

@@ -1,20 +0,0 @@
-// Declaration file for Javascript Split Software SDK
-// Project: http://www.split.io/
-
-/// <reference path="../splitio.d.ts" />
-export = JsSdk;
-
-declare module JsSdk {
-  /**
-   * Split.io sdk factory function.
-   * The settings parameter should be an object that complies with the SplitIO.INodeAsyncSettings.
-   * For more information read the corresponding article: @see {@link https://help.split.io/hc/en-us/articles/360020564931-Node-js-SDK#configuration}
-   */
-  export function SplitFactory(settings: SplitIO.INodeAsyncSettings): SplitIO.IAsyncSDK;
-  /**
-   * Split.io sdk factory function.
-   * The settings parameter should be an object that complies with the SplitIO.INodeSettings.
-   * For more information read the corresponding article: @see {@link https://help.split.io/hc/en-us/articles/360020564931-Node-js-SDK#configuration}
-   */
-  export function SplitFactory(settings: SplitIO.INodeSettings): SplitIO.ISDK;
-}