@newrelic/browser-agent 1.263.0 → 1.265.0

package/CHANGELOG.md

@@ -3,6 +3,38 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [1.265.0](https://github.com/newrelic/newrelic-browser-agent/compare/v1.264.0...v1.265.0) (2024-08-30)
+
+
+### Features
+
+* add payload size evaluation to generic events feature ([#1152](https://github.com/newrelic/newrelic-browser-agent/issues/1152)) ([65d9d11](https://github.com/newrelic/newrelic-browser-agent/commit/65d9d11172df4a3abe750c3d110b53ad65ae4319))
+* Add WebSocket wrapper and supportability metrics ([#1159](https://github.com/newrelic/newrelic-browser-agent/issues/1159)) ([7b4ab87](https://github.com/newrelic/newrelic-browser-agent/commit/7b4ab87ae28450cc97f6a84f3176e0c9f8698d76))
+* Configure Session Replay to wait for DOMContentLoaded ([#1164](https://github.com/newrelic/newrelic-browser-agent/issues/1164)) ([84bd299](https://github.com/newrelic/newrelic-browser-agent/commit/84bd29981675660d8f47a0f49235005e7be11947))
+* removing internal barrel exports ([#1158](https://github.com/newrelic/newrelic-browser-agent/issues/1158)) ([2e29a6e](https://github.com/newrelic/newrelic-browser-agent/commit/2e29a6e3c9d7a6a099b73aff59682a1e38008072))
+* Standardize Feature Buffering Behavior ([#1155](https://github.com/newrelic/newrelic-browser-agent/issues/1155)) ([d070a43](https://github.com/newrelic/newrelic-browser-agent/commit/d070a4315df35ace82e20b8b2c69044bf8504c92))
+
+
+### Bug Fixes
+
+* Ensure drain target is a valid event emitter before subscribing ([#1167](https://github.com/newrelic/newrelic-browser-agent/issues/1167)) ([187d31f](https://github.com/newrelic/newrelic-browser-agent/commit/187d31fe0fd95e75907742a7c49779683a6b040c))
+* fix duplicative log payloads on unload ([#1166](https://github.com/newrelic/newrelic-browser-agent/issues/1166)) ([52bb088](https://github.com/newrelic/newrelic-browser-agent/commit/52bb08880215c40abf080a2bf1c80a29c7ad3324))
+* relative timestamps in session trace ([#1142](https://github.com/newrelic/newrelic-browser-agent/issues/1142)) ([378b1fb](https://github.com/newrelic/newrelic-browser-agent/commit/378b1fb49ca5732e0c1b704920143b966102cfb6))
+* Updated Logging API function ([#1147](https://github.com/newrelic/newrelic-browser-agent/issues/1147)) ([bf2e980](https://github.com/newrelic/newrelic-browser-agent/commit/bf2e98056c2ad214421918fb4938a08b6213590e))
+
+
+### Security Fixes
+
+* apply obfuscation rules to session trace ([#1149](https://github.com/newrelic/newrelic-browser-agent/issues/1149)) ([fbc6f9d](https://github.com/newrelic/newrelic-browser-agent/commit/fbc6f9d31a33515e05d70e190c9c8da198fa4415))
+
+## [1.264.0](https://github.com/newrelic/newrelic-browser-agent/compare/v1.263.0...v1.264.0) (2024-08-06)
+
+
+### Features
+
+* Create generic events feature with 1,000 event limit ([#1121](https://github.com/newrelic/newrelic-browser-agent/issues/1121)) ([63ab04f](https://github.com/newrelic/newrelic-browser-agent/commit/63ab04fa9b2a0c347e175c9174f563e73767933b))
+* Report Page Actions with Generic Events Feature ([#1124](https://github.com/newrelic/newrelic-browser-agent/issues/1124)) ([0d52acf](https://github.com/newrelic/newrelic-browser-agent/commit/0d52acf9eaa9a2c170459f03e9edad25375f4c1c))
+
 ## [1.263.0](https://github.com/newrelic/newrelic-browser-agent/compare/v1.262.0...v1.263.0) (2024-07-25)
 
 

package/README.md

@@ -78,9 +78,9 @@ const options = {
 The following features may be disabled by adding `init` entries as shown above. Note that the `page_view_event` feature may not be disabled.
 
 - `ajax`
+- `generic_events`
 - `jserrors`
 - `metrics`
-- `page_action`
 - `page_view_timing`
 - `session_replay`
 - `session_trace`
@@ -125,7 +125,7 @@ The following feature modules are available for inclusion in the `features` arra
 import { Ajax } from '@newrelic/browser-agent/features/ajax';
 import { JSErrors } from '@newrelic/browser-agent/features/jserrors';
 import { Metrics } from '@newrelic/browser-agent/features/metrics';
-import { PageAction } from '@newrelic/browser-agent/features/page_action';
+import { GenericEvents } from '@newrelic/browser-agent/features/generic_events';
 import { PageViewEvent } from '@newrelic/browser-agent/features/page_view_event';
 import { PageViewTiming } from '@newrelic/browser-agent/features/page_view_timing';
 import { SessionTrace } from '@newrelic/browser-agent/features/session_trace';

package/dist/cjs/cdn/experimental.js

@@ -8,8 +8,8 @@ var _instrument4 = require("../features/jserrors/instrument");
 var _instrument5 = require("../features/ajax/instrument");
 var _instrument6 = require("../features/session_trace/instrument");
 var _instrument7 = require("../features/session_replay/instrument");
-var _instrument8 = require("../features/soft_navigations/instrument");
-var _instrument9 = require("../features/page_action/instrument");
+var _instrument8 = require("../features/generic_events/instrument");
+var _instrument9 = require("../features/soft_navigations/instrument");
 var _instrument10 = require("../features/logging/instrument");
 /*
  * Copyright 2023 New Relic Corporation. All rights reserved.
@@ -22,11 +22,7 @@ var _instrument10 = require("../features/logging/instrument");
  * It is not production ready, and is not intended to be imported or implemented in any build of the browser agent
  */
 
-// import { Instrument as InstrumentSpa } from '../features/spa/instrument'
-
 new _agent.Agent({
-  features: [_instrument5.Instrument, _instrument.Instrument, _instrument2.Instrument, _instrument6.Instrument, _instrument7.Instrument, _instrument3.Instrument, _instrument9.Instrument, _instrument4.Instrument, _instrument10.Instrument,
-  // InstrumentSpa,
-  _instrument8.Instrument],
+  features: [_instrument5.Instrument, _instrument.Instrument, _instrument2.Instrument, _instrument6.Instrument, _instrument7.Instrument, _instrument3.Instrument, _instrument4.Instrument, _instrument8.Instrument, _instrument10.Instrument, _instrument9.Instrument],
   loaderType: 'experimental'
 });

package/dist/cjs/cdn/pro.js

@@ -8,7 +8,7 @@ var _instrument4 = require("../features/jserrors/instrument");
 var _instrument5 = require("../features/ajax/instrument");
 var _instrument6 = require("../features/session_trace/instrument");
 var _instrument7 = require("../features/session_replay/instrument");
-var _instrument8 = require("../features/page_action/instrument");
+var _instrument8 = require("../features/generic_events/instrument");
 var _instrument9 = require("../features/logging/instrument");
 /**
  * @file Creates a "Pro" agent loader bundle composed of the core Agent and all available feature modules except `spa`.
@@ -16,6 +16,6 @@ var _instrument9 = require("../features/logging/instrument");
  */
 
 new _agent.Agent({
-  features: [_instrument.Instrument, _instrument2.Instrument, _instrument6.Instrument, _instrument7.Instrument, _instrument5.Instrument, _instrument3.Instrument, _instrument8.Instrument, _instrument4.Instrument, _instrument9.Instrument],
+  features: [_instrument.Instrument, _instrument2.Instrument, _instrument6.Instrument, _instrument7.Instrument, _instrument5.Instrument, _instrument3.Instrument, _instrument4.Instrument, _instrument8.Instrument, _instrument9.Instrument],
   loaderType: 'pro'
 });

package/dist/cjs/cdn/spa.js

@@ -10,14 +10,14 @@ var _instrument6 = require("../features/session_trace/instrument");
 var _instrument7 = require("../features/session_replay/instrument");
 var _instrument8 = require("../features/soft_navigations/instrument");
 var _instrument9 = require("../features/spa/instrument");
-var _instrument10 = require("../features/page_action/instrument");
+var _instrument10 = require("../features/generic_events/instrument");
 var _instrument11 = require("../features/logging/instrument");
 /**
  * @file Creates a "SPA" agent loader bundle composed of the core agent and all available feature modules.
  */
 
 new _agent.Agent({
-  features: [_instrument5.Instrument, _instrument.Instrument, _instrument2.Instrument, _instrument6.Instrument, _instrument7.Instrument, _instrument3.Instrument, _instrument10.Instrument, _instrument4.Instrument, _instrument11.Instrument, _instrument8.Instrument, _instrument9.Instrument // either the softnav or the old spa will be used (not both), but we still need to pack both to avoid dynamic import for instrument files
+  features: [_instrument5.Instrument, _instrument.Instrument, _instrument2.Instrument, _instrument6.Instrument, _instrument7.Instrument, _instrument3.Instrument, _instrument4.Instrument, _instrument10.Instrument, _instrument11.Instrument, _instrument8.Instrument, _instrument9.Instrument // either the softnav or the old spa will be used (not both), but we still need to pack both to avoid dynamic import for instrument files
   ],
   loaderType: 'spa'
 });

package/dist/cjs/common/aggregate/aggregator.js

@@ -5,7 +5,6 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.Aggregator = void 0;
 var _sharedContext = require("../context/shared-context");
-var _mapOwn = require("../util/map-own");
 /*
  * Copyright 2020 New Relic Corporation. All rights reserved.
  * SPDX-License-Identifier: Apache-2.0
@@ -36,7 +35,7 @@ class Aggregator extends _sharedContext.SharedContext {
     oldMetrics.count += metrics.count;
 
     // iterate through each new metric and merge
-    (0, _mapOwn.mapOwn)(metrics, function (key, value) {
+    Object.keys(metrics || {}).forEach(key => {
       // count is a special case handled above
       if (key === 'count') return;
       var oldMetric = oldMetrics[key];
@@ -96,7 +95,7 @@ function aggregateMetrics(newMetrics, oldMetrics) {
     count: 0
   };
   oldMetrics.count += 1;
-  (0, _mapOwn.mapOwn)(newMetrics, function (key, value) {
+  Object.entries(newMetrics || {}).forEach(([key, value]) => {
     oldMetrics[key] = updateMetric(value, oldMetrics[key]);
   });
   return oldMetrics;

package/dist/cjs/common/config/{state/configurable.js → configurable.js}

@@ -4,7 +4,7 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.getModeledObject = getModeledObject;
-var _console = require("../../util/console");
+var _console = require("../util/console");
 function getModeledObject(obj, model) {
   try {
     if (!obj || typeof obj !== 'object') return (0, _console.warn)(3);

package/dist/cjs/common/config/{state/info.js → info.js}

@@ -6,7 +6,7 @@ Object.defineProperty(exports, "__esModule", {
 exports.getInfo = getInfo;
 exports.isValid = isValid;
 exports.setInfo = setInfo;
-var _nreum = require("../../window/nreum");
+var _nreum = require("../window/nreum");
 var _configurable = require("./configurable");
 const model = {
   // preset defaults

package/dist/cjs/common/config/{state/init.js → init.js}

@@ -6,11 +6,11 @@ Object.defineProperty(exports, "__esModule", {
 exports.getConfiguration = getConfiguration;
 exports.getConfigurationValue = getConfigurationValue;
 exports.setConfiguration = setConfiguration;
-var _constants = require("../../../features/logging/constants");
-var _querySelector = require("../../dom/query-selector");
-var _constants2 = require("../../session/constants");
-var _console = require("../../util/console");
-var _nreum = require("../../window/nreum");
+var _constants = require("../../features/logging/constants");
+var _querySelector = require("../dom/query-selector");
+var _constants2 = require("../session/constants");
+var _console = require("../util/console");
+var _nreum = require("../window/nreum");
 var _configurable = require("./configurable");
 const nrMask = '[data-nr-mask]';
 const model = () => {
@@ -53,6 +53,11 @@ const model = () => {
       allowed_origins: undefined
     },
     feature_flags: [],
+    generic_events: {
+      enabled: true,
+      harvestTimeSeconds: 30,
+      autoStart: true
+    },
     harvest: {
       tooManyRequestsDelay: 60
     },
@@ -73,9 +78,7 @@ const model = () => {
     },
     obfuscate: undefined,
     page_action: {
-      enabled: true,
-      harvestTimeSeconds: 30,
-      autoStart: true
+      enabled: true
     },
     page_view_event: {
       enabled: true,

package/dist/cjs/common/config/{state/loader-config.js → loader-config.js}

@@ -5,7 +5,7 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.getLoaderConfig = getLoaderConfig;
 exports.setLoaderConfig = setLoaderConfig;
-var _nreum = require("../../window/nreum");
+var _nreum = require("../window/nreum");
 var _configurable = require("./configurable");
 const model = {
   accountID: undefined,

package/dist/cjs/common/config/{state/runtime.js → runtime.js}

@@ -6,9 +6,14 @@ Object.defineProperty(exports, "__esModule", {
 exports.getRuntime = getRuntime;
 exports.setRuntime = setRuntime;
 var _configurable = require("./configurable");
-var _nreum = require("../../window/nreum");
-var _runtime = require("../../constants/runtime");
-var _env = require("../../constants/env.npm");
+var _nreum = require("../window/nreum");
+var _runtime = require("../constants/runtime");
+var _env = require("../constants/env.npm");
+/**
+ * Module level count of harvests. This property will auto-increment each time it is accessed.
+ * @type {number}
+ */
+let harvestCount = 0;
 const readonly = {
   buildEnv: _env.BUILD_ENV,
   distMethod: _env.DIST_METHOD,
@@ -29,8 +34,8 @@ const model = {
   appMetadata: {},
   session: undefined,
   denyList: undefined,
-  harvestCount: 0,
-  timeKeeper: undefined
+  timeKeeper: undefined,
+  obfuscator: undefined
 };
 const _cache = {};
 function getRuntime(id) {
@@ -44,6 +49,12 @@ function setRuntime(id, obj) {
     ...(0, _configurable.getModeledObject)(obj, model),
     ...readonly
   };
+  if (!Object.hasOwnProperty.call(_cache[id], 'harvestCount')) {
+    // Harvest count needs to be added as a getter so the variable is updated each time it is accessed
+    Object.defineProperty(_cache[id], 'harvestCount', {
+      get: () => ++harvestCount
+    });
+  }
   const agentInst = (0, _nreum.getNREUMInitializedAgent)(id);
   if (agentInst) agentInst.runtime = _cache[id];
 }

package/dist/cjs/common/constants/agent-constants.js

@@ -0,0 +1,8 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.MAX_PAYLOAD_SIZE = exports.IDEAL_PAYLOAD_SIZE = void 0;
+const IDEAL_PAYLOAD_SIZE = exports.IDEAL_PAYLOAD_SIZE = 64000;
+const MAX_PAYLOAD_SIZE = exports.MAX_PAYLOAD_SIZE = 1000000;

package/dist/cjs/common/constants/env.cdn.js

@@ -12,7 +12,7 @@ exports.VERSION = exports.RRWEB_VERSION = exports.DIST_METHOD = exports.BUILD_EN
 /**
  * Exposes the version of the agent
  */
-const VERSION = exports.VERSION = "1.263.0";
+const VERSION = exports.VERSION = "1.265.0";
 
 /**
  * Exposes the build type of the agent

package/dist/cjs/common/constants/env.npm.js

@@ -12,7 +12,7 @@ exports.VERSION = exports.RRWEB_VERSION = exports.DIST_METHOD = exports.BUILD_EN
 /**
  * Exposes the version of the agent
  */
-const VERSION = exports.VERSION = "1.263.0";
+const VERSION = exports.VERSION = "1.265.0";
 
 /**
  * Exposes the build type of the agent

package/dist/cjs/common/constants/runtime.js

@@ -3,7 +3,7 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.supportsSendBeacon = exports.originTime = exports.loadedAsDeferredBrowserScript = exports.isiOS = exports.isWorkerScope = exports.isIE = exports.isBrowserScope = exports.initiallyHidden = exports.initialLocation = exports.iOSBelow16 = exports.globalScope = exports.ffVersion = void 0;
+exports.supportsSendBeacon = exports.originTime = exports.loadedAsDeferredBrowserScript = exports.isiOS = exports.isWorkerScope = exports.isBrowserScope = exports.initiallyHidden = exports.initialLocation = exports.iOSBelow16 = exports.globalScope = exports.ffVersion = void 0;
 var _now = require("../timing/now");
 /**
  * @file Contains constants about the environment the agent is running
@@ -42,8 +42,6 @@ const ffVersion = exports.ffVersion = (() => {
   }
   return 0;
 })();
-const isIE = exports.isIE = Boolean(isBrowserScope && window.document.documentMode); // deprecated property that only works in IE
-
 const supportsSendBeacon = exports.supportsSendBeacon = !!globalScope.navigator?.sendBeacon;
 
 /**

package/dist/cjs/common/context/shared-context.js

@@ -15,8 +15,7 @@ class SharedContext {
       if (typeof context !== 'object') return (0, _console.warn)(8);
       this.sharedContext = {};
       Object.assign(this.sharedContext, model);
-      Object.entries(context).forEach(_ref => {
-        let [key, value] = _ref;
+      Object.entries(context).forEach(([key, value]) => {
         if (Object.keys(model).includes(key)) this.sharedContext[key] = value;
       });
     } catch (err) {

package/dist/cjs/common/dispatch/global-event.js

@@ -6,8 +6,7 @@ Object.defineProperty(exports, "__esModule", {
 exports.dispatchGlobalEvent = dispatchGlobalEvent;
 var _runtime = require("../constants/runtime");
 const GLOBAL_EVENT_NAMESPACE = 'newrelic';
-function dispatchGlobalEvent() {
-  let detail = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
+function dispatchGlobalEvent(detail = {}) {
   try {
     _runtime.globalScope.dispatchEvent(new CustomEvent(GLOBAL_EVENT_NAMESPACE, {
       detail

package/dist/cjs/common/drain/drain.js

@@ -7,9 +7,9 @@ exports.deregisterDrain = deregisterDrain;
 exports.drain = drain;
 exports.registerDrain = registerDrain;
 var _contextualEe = require("../event-emitter/contextual-ee");
-var _mapOwn = require("../util/map-own");
 var _registerHandler = require("../event-emitter/register-handler");
 var _features = require("../../loaders/features/features");
+var _eventContext = require("../event-emitter/event-context");
 /*
  * Copyright 2020 New Relic Corporation. All rights reserved.
  * SPDX-License-Identifier: Apache-2.0
@@ -65,10 +65,7 @@ function curateRegistry(agentIdentifier) {
  * @param {string} featureName - A named group into which the feature's buffered events are bucketed.
  * @param {boolean} force - Whether to force the drain to occur immediately, bypassing the registry and staging logic.
  */
-function drain() {
-  let agentIdentifier = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : '';
-  let featureName = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : 'feature';
-  let force = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : false;
+function drain(agentIdentifier = '', featureName = 'feature', force = false) {
   curateRegistry(agentIdentifier);
   // If the feature for the specified agent is not in the registry, that means the instrument file was bypassed.
   // This could happen in tests, or loaders that directly import the aggregator. In these cases it is safe to
@@ -85,13 +82,9 @@ function checkCanDrainAll(agentIdentifier) {
   // Only when the event-groups for all features are ready to drain (staged) do we execute the drain. This has the effect
   // that the last feature to call drain triggers drain for all features.
   const items = Array.from(registry[agentIdentifier]);
-  if (items.every(_ref => {
-    let [key, values] = _ref;
-    return values.staged;
-  })) {
+  if (items.every(([key, values]) => values.staged)) {
     items.sort((a, b) => a[1].priority - b[1].priority);
-    items.forEach(_ref2 => {
-      let [group] = _ref2;
+    items.forEach(([group]) => {
       registry[agentIdentifier].delete(group);
       drainGroup(agentIdentifier, group);
     });
@@ -103,8 +96,7 @@ function checkCanDrainAll(agentIdentifier) {
    * the subscribed handlers for the group.
    * @param {*} group - The name of a particular feature's event "bucket".
    */
-function drainGroup(agentIdentifier, group) {
-  let activateGroup = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : true;
+function drainGroup(agentIdentifier, group, activateGroup = true) {
   const baseEE = agentIdentifier ? _contextualEe.ee.get(agentIdentifier) : _contextualEe.ee;
   const handlers = _registerHandler.registerHandler.handlers; // other storage in registerHandler
   if (baseEE.aborted || !baseEE.backlog || !handlers) return;
@@ -119,10 +111,12 @@ function drainGroup(agentIdentifier, group) {
         // eslint-disable-line no-unmodified-loop-condition
         emitEvent(bufferedEventsInGroup[i], groupHandlers);
       }
-      (0, _mapOwn.mapOwn)(groupHandlers, function (eventType, handlerRegistrationList) {
-        (0, _mapOwn.mapOwn)(handlerRegistrationList, function (i, registration) {
-          // registration is an array of: [targetEE, eventHandler]
-          registration[0].on(eventType, registration[1]);
+      Object.entries(groupHandlers).forEach(([eventType, handlerRegistrationList]) => {
+        Object.values(handlerRegistrationList || {}).forEach(registration => {
+          // registration *should* be an array of: [targetEE, eventHandler]
+          // certain browser polyfills of .values and .entries pass the prototype methods into the callback,
+          // which breaks this assumption and throws errors. So we make sure here that we only call .on() if its an actual NR EE
+          if (registration[0]?.on && registration[0]?.context() instanceof _eventContext.EventContext) registration[0].on(eventType, registration[1]);
         });
       });
     }
@@ -141,7 +135,7 @@ function drainGroup(agentIdentifier, group) {
  */
 function emitEvent(evt, groupHandlers) {
   var type = evt[1];
-  (0, _mapOwn.mapOwn)(groupHandlers[type], function (i, registration) {
+  Object.values(groupHandlers[type] || {}).forEach(registration => {
     var sourceEE = evt[0];
     var ee = registration[0];
     if (ee === sourceEE) {

package/dist/cjs/common/event-emitter/contextual-ee.js

@@ -6,7 +6,7 @@ Object.defineProperty(exports, "__esModule", {
 exports.ee = exports.contextId = void 0;
 var _nreum = require("../window/nreum");
 var _getOrSet = require("../util/get-or-set");
-var _config = require("../config/config");
+var _runtime = require("../config/runtime");
 var _eventContext = require("./event-context");
 var _bundleId = require("../ids/bundle-id");
 /*
@@ -36,7 +36,7 @@ function ee(old, debugId) {
     // We only want to check the runtime configuration for `isolatedBacklog` if the event emitter belongs to a feature.
     // For feature event emitters, the debugId will be an agentIdentifier with a length of 16. In contrast, some of our
     // tests do not namespace the event emitter with an agentID and just use the parent (which has no ID).
-    isolatedBacklog = debugId.length !== 16 ? false : (0, _config.getRuntime)(debugId).isolatedBacklog;
+    isolatedBacklog = debugId.length !== 16 ? false : (0, _runtime.getRuntime)(debugId).isolatedBacklog;
   } catch (err) {
     // Do nothing for now.
   }
@@ -129,8 +129,7 @@ function ee(old, debugId) {
 
     // do not buffer events if agent has been aborted
     if (emitter.aborted) return;
-    Object.entries(types || {}).forEach(_ref => {
-      let [_, type] = _ref;
+    Object.entries(types || {}).forEach(([_, type]) => {
       bufferGroupMap[type] = group;
       if (!(group in eventBuffer)) {
         eventBuffer[group] = [];

package/dist/cjs/common/event-listener/event-listener-opts.js

@@ -37,14 +37,10 @@ function eventListenerOpts(useCapture, abortSignal) {
 }
 
 /** Do not use this within the worker context. */
-function windowAddEventListener(event, listener) {
-  let capture = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : false;
-  let abortSignal = arguments.length > 3 ? arguments[3] : undefined;
+function windowAddEventListener(event, listener, capture = false, abortSignal) {
   window.addEventListener(event, listener, eventListenerOpts(capture, abortSignal));
 }
 /** Do not use this within the worker context. */
-function documentAddEventListener(event, listener) {
-  let capture = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : false;
-  let abortSignal = arguments.length > 3 ? arguments[3] : undefined;
+function documentAddEventListener(event, listener, capture = false, abortSignal) {
   document.addEventListener(event, listener, eventListenerOpts(capture, abortSignal));
 }

package/dist/cjs/common/harvest/harvest-scheduler.js

@@ -69,8 +69,7 @@ class HarvestScheduler extends _sharedContext.SharedContext {
     this.started = true;
     this.scheduleHarvest(initialDelay != null ? initialDelay : this.interval);
   }
-  stopTimer() {
-    let permanently = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : false;
+  stopTimer(permanently = false) {
     this.aborted = permanently; // stopping permanently is same as aborting, but this function also cleans up the setTimeout loop
     this.started = false;
     if (this.timeoutHandle) {

package/dist/cjs/common/harvest/harvest.js

@@ -8,14 +8,14 @@ var _encode = require("../url/encode");
 var _stringify = require("../util/stringify");
 var submitData = _interopRequireWildcard(require("../util/submit-data"));
 var _location = require("../url/location");
-var _config = require("../config/config");
+var _info = require("../config/info");
+var _init = require("../config/init");
+var _runtime = require("../config/runtime");
 var _cleanUrl = require("../url/clean-url");
 var _eventListenerOpts = require("../event-listener/event-listener-opts");
-var _obfuscate = require("../util/obfuscate");
-var _traverse = require("../util/traverse");
 var _sharedContext = require("../context/shared-context");
 var _env = require("../constants/env.npm");
-var _runtime = require("../constants/runtime");
+var _runtime2 = require("../constants/runtime");
 var _console = require("../util/console");
 var _now = require("../timing/now");
 function _getRequireWildcardCache(e) { if ("function" != typeof WeakMap) return null; var r = new WeakMap(), t = new WeakMap(); return (_getRequireWildcardCache = function (e) { return e ? t : r; })(e); }
@@ -38,8 +38,8 @@ class Harvest extends _sharedContext.SharedContext {
   constructor(parent) {
     super(parent); // gets any allowed properties from the parent and stores them in `sharedContext`
 
-    this.tooManyRequestsDelay = (0, _config.getConfigurationValue)(this.sharedContext.agentIdentifier, 'harvest.tooManyRequestsDelay') || 60;
-    this.obfuscator = new _obfuscate.Obfuscator(this.sharedContext);
+    this.tooManyRequestsDelay = (0, _init.getConfigurationValue)(this.sharedContext.agentIdentifier, 'harvest.tooManyRequestsDelay') || 60;
+    this.obfuscator = (0, _runtime.getRuntime)(this.sharedContext.agentIdentifier).obfuscator;
     this._events = {};
   }
 
@@ -49,8 +49,7 @@ class Harvest extends _sharedContext.SharedContext {
    * Note: Used by page_action
    * @param {NetworkSendSpec} spec Specification for sending data
    */
-  sendX() {
-    let spec = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
+  sendX(spec = {}) {
     const submitMethod = submitData.getSubmitMethod({
       isFinalHarvest: spec.opts?.unload
     });
@@ -59,7 +58,7 @@ class Harvest extends _sharedContext.SharedContext {
       isFinalHarvest: spec.opts?.unload === true
     };
     const payload = this.createPayload(spec.endpoint, options);
-    const caller = this.obfuscator.shouldObfuscate() ? this.obfuscateAndSend.bind(this) : this._send.bind(this);
+    const caller = this._send.bind(this);
     return caller({
       ...spec,
       payload,
@@ -71,31 +70,11 @@ class Harvest extends _sharedContext.SharedContext {
    * Initiate a harvest call.
    * @param {NetworkSendSpec} spec Specification for sending data
    */
-  send() {
-    let spec = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
-    const caller = this.obfuscator.shouldObfuscate() ? this.obfuscateAndSend.bind(this) : this._send.bind(this);
+  send(spec = {}) {
+    const caller = this._send.bind(this);
     return caller(spec);
   }
 
-  /**
-   * Apply obfuscation rules to the payload and then initial the harvest network call.
-   * @param {NetworkSendSpec} spec Specification for sending data
-   */
-  obfuscateAndSend() {
-    var _this = this;
-    let spec = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
-    const {
-      payload = {}
-    } = spec;
-    (0, _traverse.applyFnToProps)(payload, function () {
-      return _this.obfuscator.obfuscateString(...arguments);
-    }, 'string', ['e']);
-    return this._send({
-      ...spec,
-      payload
-    });
-  }
-
   /**
    * Initiate a harvest call. Typically used by `sendX` and `send` methods or called directly
    * for raw network calls.
@@ -103,20 +82,19 @@ class Harvest extends _sharedContext.SharedContext {
    * @returns {boolean} True if the network call succeeded. For final harvest calls, the return
    * value should not be relied upon because network calls will be made asynchronously.
    */
-  _send(_ref) {
-    let {
-      endpoint,
-      payload = {},
-      opts = {},
-      submitMethod,
-      cbFinished,
-      customUrl,
-      raw,
-      includeBaseParams = true
-    } = _ref;
-    const info = (0, _config.getInfo)(this.sharedContext.agentIdentifier);
+  _send({
+    endpoint,
+    payload = {},
+    opts = {},
+    submitMethod,
+    cbFinished,
+    customUrl,
+    raw,
+    includeBaseParams = true
+  }) {
+    const info = (0, _info.getInfo)(this.sharedContext.agentIdentifier);
     if (!info.errorBeacon) return false;
-    const agentRuntime = (0, _config.getRuntime)(this.sharedContext.agentIdentifier);
+    const agentRuntime = (0, _runtime.getRuntime)(this.sharedContext.agentIdentifier);
     let {
       body,
       qs
@@ -130,7 +108,7 @@ class Harvest extends _sharedContext.SharedContext {
       }
       return false;
     }
-    const init = (0, _config.getConfiguration)(this.sharedContext.agentIdentifier);
+    const init = (0, _init.getConfiguration)(this.sharedContext.agentIdentifier);
     const protocol = init.ssl === false ? 'http' : 'https';
     const perceviedBeacon = init.proxy.beacon || info.errorBeacon;
     const endpointURLPart = endpoint !== 'rum' ? "/".concat(endpoint) : '';
@@ -171,11 +149,11 @@ class Harvest extends _sharedContext.SharedContext {
     /* Since workers don't support sendBeacon right now, they can only use XHR method.
         Because they still do permit synch XHR, the idea is that at final harvest time (worker is closing),
         we just make a BLOCKING request--trivial impact--with the remaining data as a temp fill-in for sendBeacon.
-       Following the removal of img-element method, IE will also use sync XHR on page dismissal to ensure final analytics are sent. */
+       Following the removal of img-element method. */
     let result = submitMethod({
       url: fullUrl,
       body,
-      sync: opts.unload && (_runtime.isWorkerScope || _runtime.isIE),
+      sync: opts.unload && _runtime2.isWorkerScope,
       headers
     });
     if (!opts.unload && cbFinished && submitMethod === submitData.xhr) {
@@ -201,17 +179,14 @@ class Harvest extends _sharedContext.SharedContext {
         cbFinished(cbResult);
       }, (0, _eventListenerOpts.eventListenerOpts)(false));
     }
-    const runtime = (0, _config.getRuntime)(this.sharedContext.agentIdentifier);
-    runtime.harvestCount++;
     return result;
   }
 
   // The stuff that gets sent every time.
   baseQueryString(qs, endpoint) {
-    const runtime = (0, _config.getRuntime)(this.sharedContext.agentIdentifier);
-    const info = (0, _config.getInfo)(this.sharedContext.agentIdentifier);
-    const location = (0, _cleanUrl.cleanURL)((0, _location.getLocation)());
-    const ref = this.obfuscator.shouldObfuscate() ? this.obfuscator.obfuscateString(location) : location;
+    const runtime = (0, _runtime.getRuntime)(this.sharedContext.agentIdentifier);
+    const info = (0, _info.getInfo)(this.sharedContext.agentIdentifier);
+    const ref = this.obfuscator.obfuscateString((0, _cleanUrl.cleanURL)((0, _location.getLocation)()));
     const hr = runtime?.session?.state.sessionReplayMode === 1 && endpoint !== 'jserrors';
     const qps = ['a=' + info.applicationID, (0, _encode.param)('sa', info.sa ? '' + info.sa : ''), (0, _encode.param)('v', _env.VERSION), transactionNameParam(info), (0, _encode.param)('ct', runtime.customTransaction), '&rst=' + (0, _now.now)(), '&ck=0',
     // ck param DEPRECATED - still expected by backend
@@ -262,13 +237,11 @@ class Harvest extends _sharedContext.SharedContext {
    * @param {HarvestPayload} payload Payload to be sent to the endpoint.
    * @returns {HarvestPayload} Cleaned payload payload to be sent to the endpoint.
    */
-  cleanPayload() {
-    let payload = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
+  cleanPayload(payload = {}) {
     const clean = input => {
       if (typeof Uint8Array !== 'undefined' && input instanceof Uint8Array || Array.isArray(input)) return input;
       if (typeof input === 'string') return input.length > 0 ? input : null;
-      return Object.entries(input || {}).reduce((accumulator, _ref2) => {
-        let [key, value] = _ref2;
+      return Object.entries(input || {}).reduce((accumulator, [key, value]) => {
         if (typeof value === 'number' || typeof value === 'string' && value.length > 0 || typeof value === 'object' && Object.keys(value || {}).length > 0) {
           accumulator[key] = value;
         }