@schibsted/pulse-sdk 8.0.0-rc.15 → 8.0.0-rc.16

package/dist/cjs/Tracker.js

@@ -106,6 +106,7 @@ const anonymousEvents = {
     trackerEvent: events_anonymous_node_1.anonymousTrackerEvent,
     engagementEvent: events_anonymous_node_1.anonymousEngagementEvent,
 };
+const ANONYMOUS_EVENT_FORBIDDEN_FIELDS = ['session', 'actor', 'device', 'location', 'pageViewId'];
 /**
  * @typeParam T Type of event to track
  * @param name
@@ -464,11 +465,11 @@ class Tracker {
      */
     async trackAnonymous(eventType, input, options) {
         // We must create the event as soon as possible, as it defines the creationDate.
-        const newEvent = this.createAnonymousEvent(input);
+        const newEvent = (0, utils_1.omit)(this.createAnonymousEvent(input), ANONYMOUS_EVENT_FORBIDDEN_FIELDS);
         // Remote config values ----------------
         await this._applyRemoteConfigEndpoints();
         // Resolve all available properties ----------------
-        const resolvedOutput = await Promise.all([this.resolveTrackInput(newEvent), this.resolveTrackGlobals()]);
+        const resolvedOutput = await Promise.all([this.resolveTrackInput(newEvent), this.resolveAnonymousTrackGlobals()]);
         let eventOutput = resolvedOutput.reduce((acc, p) => (0, utils_1.pulseMerge)(acc, p), {});
         eventOutput = this.applyAnonymousEventTypeDefaults(eventOutput, eventType);
         // Leave Events ----------------
@@ -482,6 +483,7 @@ class Tracker {
             eventInput: eventOutput,
             sdkConfig: this.state,
         });
+        eventOutput = (0, utils_1.omit)(eventOutput, ANONYMOUS_EVENT_FORBIDDEN_FIELDS);
         if (options?.bookmarkKey) {
             this.bookmark.set(options.bookmarkKey, eventOutput);
         }
@@ -2307,8 +2309,9 @@ class Tracker {
     /**
      * @private
      */
-    async resolveTrackGlobals() {
-        return (0, utils_1.evaluateAndFlatten)({ ...this.builders });
+    async resolveAnonymousTrackGlobals() {
+        // Omit before evaluation so forbidden promise/function builders cannot delay or fail anonymous tracking.
+        return (0, utils_1.evaluateAndFlatten)((0, utils_1.omit)(this.builders, ANONYMOUS_EVENT_FORBIDDEN_FIELDS));
     }
     /**
      * Send all anonymous events that are in the internal anonymous event queue.

package/dist/cjs/version.js

@@ -2,4 +2,4 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.SDK_VERSION = void 0;
 // This file is auto-generated by bin/generate-version.js. Do not edit manually.
-exports.SDK_VERSION = '8.0.0-rc.15';
+exports.SDK_VERSION = '8.0.0-rc.16';

package/dist/ejs/Tracker.js

@@ -21,7 +21,7 @@ import * as config from './remoteConfig';
 import { buildAnonymousViewUIElementEvent, buildCompletedHealthActionEvent, buildEngagementAffiliateEvent, buildEngagementAudioEvent, buildEngagementFormEvent, buildEngagementHealthUIElementEvent, buildEngagementOfferEvent, buildEngagementTeaserEvent, buildEngagementTvModuleEvent, buildEngagementUIElementEvent, buildEngagementVideoAdEvent, buildEngagementVideoEvent, buildEngagementWidgetEvent, buildImpressionAdSlotEvent, buildImpressionAffiliateEvent, buildImpressionFormEvent, buildImpressionHealthUIElementEvent, buildImpressionOfferEvent, buildImpressionPlayerEvent, buildImpressionTeaserEvent, buildImpressionUIElementEvent, buildImpressionWidgetEvent, buildLeaveArticleEvent, buildLeaveAudioPageEvent, buildLeaveErrorEvent, buildLeaveFrontpageEvent, buildLeaveLandingpageEvent, buildLeaveListingEvent, buildLeaveLockedArticleEvent, buildLeaveLockedAudioPageEvent, buildLeaveLockedVideoPageEvent, buildLeavePageEvent, buildLeaveSportsPageEvent, buildLeaveVideoPageEvent, buildLeaveWeatherEvent, buildViewArticleEvent, buildViewAudioPageEvent, buildViewErrorEvent, buildViewFrontpageEvent, buildViewHealthPageEvent, buildViewLandingpageEvent, buildViewListingEvent, buildViewLockedArticleEvent, buildViewLockedAudioPageEvent, buildViewLockedVideoPageEvent, buildViewPageEvent, buildViewSportsPageEvent, buildViewTitleEvent, buildViewVideoPageEvent, buildViewWeatherEvent, } from './tpaas-builders';
 import { LeaveArticle, LeaveAudioPage, LeaveError, LeaveFrontpage, LeaveLandingpage, LeaveListing, LeaveLockedArticle, LeaveLockedAudioPage, LeaveLockedVideoPage, LeavePage, LeaveSportsPage, LeaveVideoPage, LeaveWeather, ViewArticle, ViewAudioPage, ViewError, ViewFrontpage, ViewHealthPage, ViewLandingpage, ViewListing, ViewLockedArticle, ViewLockedAudioPage, ViewLockedVideoPage, ViewPage, ViewSportsPage, ViewTitle, ViewVideoPage, ViewWeather, } from './tpaas-schemas';
 import { startProxyProviderListener } from './tracker-proxy/proxy-provider';
-import { evaluateAndFlatten, isBrowser, isFunction, isPromise, isString, pulseMerge, throttle } from './utils';
+import { evaluateAndFlatten, isBrowser, isFunction, isPromise, isString, omit, pulseMerge, throttle } from './utils';
 import { SDK_VERSION } from './version';
 import { VisibilityObserver } from './visibility-observer';
 import warnOnce from './warnOnce';
@@ -63,6 +63,7 @@ const anonymousEvents = {
     trackerEvent: anonymousTrackerEvent,
     engagementEvent: anonymousEngagementEvent,
 };
+const ANONYMOUS_EVENT_FORBIDDEN_FIELDS = ['session', 'actor', 'device', 'location', 'pageViewId'];
 /**
  * @typeParam T Type of event to track
  * @param name
@@ -421,11 +422,11 @@ export default class Tracker {
      */
     async trackAnonymous(eventType, input, options) {
         // We must create the event as soon as possible, as it defines the creationDate.
-        const newEvent = this.createAnonymousEvent(input);
+        const newEvent = omit(this.createAnonymousEvent(input), ANONYMOUS_EVENT_FORBIDDEN_FIELDS);
         // Remote config values ----------------
         await this._applyRemoteConfigEndpoints();
         // Resolve all available properties ----------------
-        const resolvedOutput = await Promise.all([this.resolveTrackInput(newEvent), this.resolveTrackGlobals()]);
+        const resolvedOutput = await Promise.all([this.resolveTrackInput(newEvent), this.resolveAnonymousTrackGlobals()]);
         let eventOutput = resolvedOutput.reduce((acc, p) => pulseMerge(acc, p), {});
         eventOutput = this.applyAnonymousEventTypeDefaults(eventOutput, eventType);
         // Leave Events ----------------
@@ -439,6 +440,7 @@ export default class Tracker {
             eventInput: eventOutput,
             sdkConfig: this.state,
         });
+        eventOutput = omit(eventOutput, ANONYMOUS_EVENT_FORBIDDEN_FIELDS);
         if (options?.bookmarkKey) {
             this.bookmark.set(options.bookmarkKey, eventOutput);
         }
@@ -2264,8 +2266,9 @@ export default class Tracker {
     /**
      * @private
      */
-    async resolveTrackGlobals() {
-        return evaluateAndFlatten({ ...this.builders });
+    async resolveAnonymousTrackGlobals() {
+        // Omit before evaluation so forbidden promise/function builders cannot delay or fail anonymous tracking.
+        return evaluateAndFlatten(omit(this.builders, ANONYMOUS_EVENT_FORBIDDEN_FIELDS));
     }
     /**
      * Send all anonymous events that are in the internal anonymous event queue.

package/dist/ejs/version.js

@@ -1,2 +1,2 @@
 // This file is auto-generated by bin/generate-version.js. Do not edit manually.
-export const SDK_VERSION = '8.0.0-rc.15';
+export const SDK_VERSION = '8.0.0-rc.16';

package/dist/types/Tracker.d.ts

@@ -1334,7 +1334,7 @@ export default class Tracker {
     /**
      * @private
      */
-    private resolveTrackGlobals;
+    private resolveAnonymousTrackGlobals;
     /**
      * Send all anonymous events that are in the internal anonymous event queue.
      *

package/dist/types/version.d.ts

@@ -1 +1 @@
-export declare const SDK_VERSION = "8.0.0-rc.15";
+export declare const SDK_VERSION = "8.0.0-rc.16";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@schibsted/pulse-sdk",
-  "version": "8.0.0-rc.15",
+  "version": "8.0.0-rc.16",
   "description": "Node.js / Browser Pulse SDK",
   "author": "Schibsted Data&Tech",
   "license": "UNLICENSED",