@newrelic/browser-agent 1.258.0 → 1.258.1

package/CHANGELOG.md

@@ -3,6 +3,17 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [1.258.1](https://github.com/newrelic/newrelic-browser-agent/compare/v1.258.0...v1.258.1) (2024-05-07)
+
+
+### Bug Fixes
+
+* Exclude data url requests as captured AJAX events ([#1012](https://github.com/newrelic/newrelic-browser-agent/issues/1012)) ([2a3fa57](https://github.com/newrelic/newrelic-browser-agent/commit/2a3fa57da1f389e6eabae2c92686e25285fc6cd3))
+* Improve Error Casting ([#1014](https://github.com/newrelic/newrelic-browser-agent/issues/1014)) ([d1dd20c](https://github.com/newrelic/newrelic-browser-agent/commit/d1dd20ce526ddb697962f695fbb5915410474987))
+* Remove API start()'s features param ([#1009](https://github.com/newrelic/newrelic-browser-agent/issues/1009)) ([38a502b](https://github.com/newrelic/newrelic-browser-agent/commit/38a502b08b4735259e7f5b863b0e5e5361c075b6))
+* Run inside cross-origin iframes for firefox/safari ([#1015](https://github.com/newrelic/newrelic-browser-agent/issues/1015)) ([6a4a73d](https://github.com/newrelic/newrelic-browser-agent/commit/6a4a73d72b056177268f8619a3a3b7810a7a2c79))
+* Session trace nodes de-duplication ([#1008](https://github.com/newrelic/newrelic-browser-agent/issues/1008)) ([44f229e](https://github.com/newrelic/newrelic-browser-agent/commit/44f229e4d35cd468bfe29b1796be8031bb9c72ff))
+
 ## [1.258.0](https://github.com/newrelic/newrelic-browser-agent/compare/v1.257.0...v1.258.0) (2024-04-29)
 
 

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.258.0";
+const VERSION = exports.VERSION = "1.258.1";
 
 /**
  * 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.258.0";
+const VERSION = exports.VERSION = "1.258.1";
 
 /**
  * Exposes the build type of the agent

package/dist/cjs/common/deny-list/deny-list.js

@@ -3,6 +3,7 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
+exports.hasUndefinedHostname = hasUndefinedHostname;
 exports.setDenyList = setDenyList;
 exports.shouldCollectEvent = shouldCollectEvent;
 /** An array of filter objects {hostname, pathname} for identifying XHR events to be excluded from collection.
@@ -17,14 +18,8 @@ var denyList = [];
  * @returns {boolean} `true` if request does not match any entries of {@link denyList|deny list}; else `false`
  */
 function shouldCollectEvent(params) {
-  if (denyList.length === 0) {
-    return true;
-  }
-
-  // XHR requests with an undefined hostname (e.g., data URLs) should not be collected.
-  if (params.hostname === undefined) {
-    return false;
-  }
+  if (hasUndefinedHostname(params)) return false;
+  if (denyList.length === 0) return true;
   for (var i = 0; i < denyList.length; i++) {
     var parsed = denyList[i];
     if (parsed.hostname === '*') {
@@ -36,6 +31,9 @@ function shouldCollectEvent(params) {
   }
   return true;
 }
+function hasUndefinedHostname(params) {
+  return params.hostname === undefined; // requests with an undefined hostname (e.g., data URLs) should not be collected.
+}
 
 /**
  * Initializes the {@link denyList|XHR deny list} by extracting hostname and pathname from an array of filter strings.

package/dist/cjs/common/vitals/time-to-first-byte.js

@@ -9,7 +9,15 @@ var _constants = require("./constants");
 var _vitalMetric = require("./vital-metric");
 var _attribution = require("web-vitals/attribution");
 const timeToFirstByte = exports.timeToFirstByte = new _vitalMetric.VitalMetric(_constants.VITAL_NAMES.TIME_TO_FIRST_BYTE);
-if (_runtime.isBrowserScope && typeof PerformanceNavigationTiming !== 'undefined' && !_runtime.isiOS) {
+
+/**
+ * onTTFB is not supported in the following scenarios:
+ * - in a non-browser scope
+ * - in browsers that do not support PerformanceNavigationTiming API
+ * - in an iOS browser
+ * - cross-origin iframes specifically in firefox and safari
+ */
+if (_runtime.isBrowserScope && typeof PerformanceNavigationTiming !== 'undefined' && !_runtime.isiOS && window === window.parent) {
   (0, _attribution.onTTFB)(_ref => {
     let {
       value,

package/dist/cjs/features/ajax/instrument/index.js

@@ -19,6 +19,7 @@ var _constants = require("../constants");
 var _features = require("../../../loaders/features/features");
 var _constants2 = require("../../metrics/constants");
 var _now = require("../../../common/timing/now");
+var _denyList = require("../../../common/deny-list/deny-list");
 /*
  * Copyright 2020 New Relic Corporation. All rights reserved.
  * SPDX-License-Identifier: Apache-2.0
@@ -315,17 +316,17 @@ function subscribeToEvents(agentIdentifier, ee, handler, dt) {
   // eslint-disable-next-line handle-callback-err
   function onFetchDone(_, res) {
     this.endTime = (0, _now.now)();
-    if (!this.params) {
-      this.params = {};
-    }
+    if (!this.params) this.params = {};
+    if ((0, _denyList.hasUndefinedHostname)(this.params)) return; // don't bother with fetch to url with no hostname
+
     this.params.status = res ? res.status : 0;
 
     // convert rxSize to a number
-    var responseSize;
+    let responseSize;
     if (typeof this.rxSize === 'string' && this.rxSize.length > 0) {
       responseSize = +this.rxSize;
     }
-    var metrics = {
+    const metrics = {
       txSize: this.txSize,
       rxSize: responseSize,
       duration: (0, _now.now)() - this.startTime
@@ -335,14 +336,16 @@ function subscribeToEvents(agentIdentifier, ee, handler, dt) {
 
   // Create report for XHR request that has finished
   function end(xhr) {
-    var params = this.params;
-    var metrics = this.metrics;
+    const params = this.params;
+    const metrics = this.metrics;
     if (this.ended) return;
     this.ended = true;
-    for (var i = 0; i < handlersLen; i++) {
+    for (let i = 0; i < handlersLen; i++) {
       xhr.removeEventListener(handlers[i], this.listener, false);
     }
     if (params.aborted) return;
+    if ((0, _denyList.hasUndefinedHostname)(params)) return; // don't bother with XHR of url with no hostname
+
     metrics.duration = (0, _now.now)() - this.startTime;
     if (!this.loadCaptureCalled && xhr.readyState === 4) {
       captureXhrData(this, xhr);

package/dist/cjs/features/jserrors/aggregate/index.js

@@ -199,13 +199,17 @@ class Aggregate extends _aggregateBase.AggregateBase {
 
     // Trace sends the error in its payload, and both trace & replay simply listens for any error to occur.
     const jsErrorEvent = [type, bucketHash, params, newMetrics, customAttributes];
-    (0, _handle.handle)('errorAgg', jsErrorEvent, undefined, _features.FEATURE_NAMES.sessionTrace, this.ee);
+    (0, _handle.handle)('trace-jserror', jsErrorEvent, undefined, _features.FEATURE_NAMES.sessionTrace, this.ee);
     // still send EE events for other features such as above, but stop this one from aggregating internal data
     if (this.blocked) return;
+    if (err.__newrelic?.[this.agentIdentifier]) {
+      params._interactionId = err.__newrelic[this.agentIdentifier].interactionId;
+      params._interactionNodeId = err.__newrelic[this.agentIdentifier].interactionNodeId;
+    }
     const softNavInUse = Boolean((0, _nreum.getNREUMInitializedAgent)(this.agentIdentifier)?.features[_features.FEATURE_NAMES.softNav]);
     // Note: the following are subject to potential race cond wherein if the other feature aren't fully initialized, it'll be treated as there being no associated interaction.
     // They each will also tack on their respective properties to the params object as part of the decision flow.
-    if (softNavInUse) (0, _handle.handle)('jserror', [params, time], undefined, _features.FEATURE_NAMES.softNav, this.ee);else (0, _handle.handle)('errorAgg', jsErrorEvent, undefined, _features.FEATURE_NAMES.spa, this.ee);
+    if (softNavInUse) (0, _handle.handle)('jserror', [params, time], undefined, _features.FEATURE_NAMES.softNav, this.ee);else (0, _handle.handle)('spa-jserror', jsErrorEvent, undefined, _features.FEATURE_NAMES.spa, this.ee);
     if (params.browserInteractionId && !params._softNavFinished) {
       // hold onto the error until the in-progress interaction is done, eithered saved or discarded
       this.bufferedErrorsUnderSpa[params.browserInteractionId] ??= [];

package/dist/cjs/features/jserrors/instrument/index.js

@@ -10,10 +10,9 @@ var _constants = require("../constants");
 var _features = require("../../../loaders/features/features");
 var _runtime = require("../../../common/constants/runtime");
 var _eventListenerOpts = require("../../../common/event-listener/event-listener-opts");
-var _stringify = require("../../../common/util/stringify");
-var _uncaughtError = require("./uncaught-error");
 var _now = require("../../../common/timing/now");
 var _constants2 = require("../../session_replay/constants");
+var _castError = require("../shared/cast-error");
 /*
  * Copyright 2020 New Relic Corporation. All rights reserved.
  * SPDX-License-Identifier: Apache-2.0
@@ -21,7 +20,6 @@ var _constants2 = require("../../session_replay/constants");
 
 class Instrument extends _instrumentBase.InstrumentBase {
   static featureName = _constants.FEATURE_NAME;
-  #seenErrors = new Set();
   #replayRunning = false;
   constructor(agentIdentifier, aggregator) {
     let auto = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : true;
@@ -30,38 +28,22 @@ class Instrument extends _instrumentBase.InstrumentBase {
       // this try-catch can be removed when IE11 is completely unsupported & gone
       this.removeOnAbort = new AbortController();
     } catch (e) {}
-
-    // Capture function errors early in case the spa feature is loaded
-    this.ee.on('fn-err', (args, obj, error) => {
-      if (!this.abortHandler || this.#seenErrors.has(error)) return;
-      this.#seenErrors.add(error);
-      (0, _handle.handle)('err', [this.#castError(error), (0, _now.now)()], undefined, _features.FEATURE_NAMES.jserrors, this.ee);
-    });
     this.ee.on('internal-error', error => {
       if (!this.abortHandler) return;
-      (0, _handle.handle)('ierr', [this.#castError(error), (0, _now.now)(), true, {}, this.#replayRunning], undefined, _features.FEATURE_NAMES.jserrors, this.ee);
+      (0, _handle.handle)('ierr', [(0, _castError.castError)(error), (0, _now.now)(), true, {}, this.#replayRunning], undefined, _features.FEATURE_NAMES.jserrors, this.ee);
     });
     this.ee.on(_constants2.SR_EVENT_EMITTER_TYPES.REPLAY_RUNNING, isRunning => {
       this.#replayRunning = isRunning;
     });
     _runtime.globalScope.addEventListener('unhandledrejection', promiseRejectionEvent => {
       if (!this.abortHandler) return;
-      (0, _handle.handle)('err', [this.#castPromiseRejectionEvent(promiseRejectionEvent), (0, _now.now)(), false, {
+      (0, _handle.handle)('err', [(0, _castError.castPromiseRejectionEvent)(promiseRejectionEvent), (0, _now.now)(), false, {
         unhandledPromiseRejection: 1
       }, this.#replayRunning], undefined, _features.FEATURE_NAMES.jserrors, this.ee);
     }, (0, _eventListenerOpts.eventListenerOpts)(false, this.removeOnAbort?.signal));
     _runtime.globalScope.addEventListener('error', errorEvent => {
       if (!this.abortHandler) return;
-
-      /**
-       * If the spa feature is loaded, errors may already have been captured in the `fn-err` listener above.
-       * This ensures those errors are not captured twice.
-       */
-      if (this.#seenErrors.has(errorEvent.error)) {
-        this.#seenErrors.delete(errorEvent.error);
-        return;
-      }
-      (0, _handle.handle)('err', [this.#castErrorEvent(errorEvent), (0, _now.now)(), false, {}, this.#replayRunning], undefined, _features.FEATURE_NAMES.jserrors, this.ee);
+      (0, _handle.handle)('err', [(0, _castError.castErrorEvent)(errorEvent), (0, _now.now)(), false, {}, this.#replayRunning], undefined, _features.FEATURE_NAMES.jserrors, this.ee);
     }, (0, _eventListenerOpts.eventListenerOpts)(false, this.removeOnAbort?.signal));
     this.abortHandler = this.#abort; // we also use this as a flag to denote that the feature is active or on and handling errors
     this.importAggregator();
@@ -70,72 +52,7 @@ class Instrument extends _instrumentBase.InstrumentBase {
   /** Restoration and resource release tasks to be done if JS error loader is being aborted. Unwind changes to globals. */
   #abort() {
     this.removeOnAbort?.abort();
-    this.#seenErrors.clear();
     this.abortHandler = undefined; // weakly allow this abort op to run only once
   }
-
-  /**
-   * Any value can be used with the `throw` keyword. This function ensures that the value is
-   * either a proper Error instance or attempts to convert it to an UncaughtError instance.
-   * @param {any} error The value thrown
-   * @returns {Error|UncaughtError} The converted error instance
-   */
-  #castError(error) {
-    if (error instanceof Error) {
-      return error;
-    }
-
-    /**
-     * The thrown value may contain a message property. If it does, try to treat the thrown
-     * value as an Error-like object.
-     */
-    if (typeof error?.message !== 'undefined') {
-      return new _uncaughtError.UncaughtError(error.message, error.filename || error.sourceURL, error.lineno || error.line, error.colno || error.col);
-    }
-    return new _uncaughtError.UncaughtError(typeof error === 'string' ? error : (0, _stringify.stringify)(error));
-  }
-
-  /**
-   * Attempts to convert a PromiseRejectionEvent object to an Error object
-   * @param {PromiseRejectionEvent} unhandledRejectionEvent The unhandled promise rejection event
-   * @returns {Error} An Error object with the message as the casted reason
-   */
-  #castPromiseRejectionEvent(promiseRejectionEvent) {
-    let prefix = 'Unhandled Promise Rejection: ';
-    if (promiseRejectionEvent?.reason instanceof Error) {
-      try {
-        promiseRejectionEvent.reason.message = prefix + promiseRejectionEvent.reason.message;
-        return promiseRejectionEvent.reason;
-      } catch (e) {
-        return promiseRejectionEvent.reason;
-      }
-    }
-    if (typeof promiseRejectionEvent.reason === 'undefined') return new _uncaughtError.UncaughtError(prefix);
-    const error = this.#castError(promiseRejectionEvent.reason);
-    error.message = prefix + error.message;
-    return error;
-  }
-
-  /**
-   * Attempts to convert an ErrorEvent object to an Error object
-   * @param {ErrorEvent} errorEvent The error event
-   * @returns {Error|UncaughtError} The error event converted to an Error object
-   */
-  #castErrorEvent(errorEvent) {
-    if (errorEvent.error instanceof SyntaxError && !/:\d+$/.test(errorEvent.error.stack?.trim())) {
-      const error = new _uncaughtError.UncaughtError(errorEvent.message, errorEvent.filename, errorEvent.lineno, errorEvent.colno);
-      error.name = SyntaxError.name;
-      return error;
-    }
-    if (errorEvent.error instanceof Error) {
-      return errorEvent.error;
-    }
-
-    /**
-     * Older browsers do not contain the `error` property on the ErrorEvent instance.
-     * https://caniuse.com/mdn-api_errorevent_error
-     */
-    return new _uncaughtError.UncaughtError(errorEvent.message, errorEvent.filename, errorEvent.lineno, errorEvent.colno);
-  }
 }
 exports.Instrument = Instrument;

package/dist/cjs/features/jserrors/shared/cast-error.js

@@ -0,0 +1,66 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.castError = castError;
+exports.castErrorEvent = castErrorEvent;
+exports.castPromiseRejectionEvent = castPromiseRejectionEvent;
+var _uncaughtError = require("./uncaught-error");
+/**
+   * Any value can be used with the `throw` keyword. This function ensures that the value is
+   * either a proper Error instance or attempts to convert it to an UncaughtError instance.
+   * @param {any} error The value thrown
+   * @returns {Error|UncaughtError} The converted error instance
+   */
+function castError(error) {
+  /** Sometimes a browser can emit an error object with no stack */
+  if (canTrustError(error)) {
+    return error;
+  }
+
+  /**
+     * The thrown value may contain a message property. If it does, try to treat the thrown
+     * value as an Error-like object.
+     */
+  return new _uncaughtError.UncaughtError(error?.message !== undefined ? error.message : error, error?.filename || error?.sourceURL, error?.lineno || error?.line, error?.colno || error?.col, error?.__newrelic);
+}
+
+/**
+   * Attempts to convert a PromiseRejectionEvent object to an Error object
+   * @param {PromiseRejectionEvent} unhandledRejectionEvent The unhandled promise rejection event
+   * @returns {Error} An Error object with the message as the casted reason
+   */
+function castPromiseRejectionEvent(promiseRejectionEvent) {
+  let prefix = 'Unhandled Promise Rejection';
+  if (canTrustError(promiseRejectionEvent?.reason)) {
+    try {
+      promiseRejectionEvent.reason.message = prefix + ': ' + promiseRejectionEvent.reason.message;
+      return castError(promiseRejectionEvent.reason);
+    } catch (e) {
+      return castError(promiseRejectionEvent.reason);
+    }
+  }
+  if (typeof promiseRejectionEvent.reason === 'undefined') return castError(prefix);
+  const error = castError(promiseRejectionEvent.reason);
+  error.message = prefix + ': ' + error?.message;
+  return error;
+}
+
+/**
+   * Attempts to convert an ErrorEvent object to an Error object
+   * @param {ErrorEvent} errorEvent The error event
+   * @returns {Error|UncaughtError} The error event converted to an Error object
+   */
+function castErrorEvent(errorEvent) {
+  if (errorEvent.error instanceof SyntaxError && !/:\d+$/.test(errorEvent.error.stack?.trim())) {
+    const error = new _uncaughtError.UncaughtError(errorEvent.message, errorEvent.filename, errorEvent.lineno, errorEvent.colno, errorEvent.error.__newrelic);
+    error.name = SyntaxError.name;
+    return error;
+  }
+  if (canTrustError(errorEvent.error)) return errorEvent.error;
+  return castError(errorEvent);
+}
+function canTrustError(error) {
+  return error instanceof Error && !!error.stack;
+}

package/dist/cjs/features/jserrors/{instrument → shared}/uncaught-error.js

@@ -4,6 +4,7 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.UncaughtError = void 0;
+var _stringify = require("../../../common/util/stringify");
 /**
  * Represents an uncaught non Error type error. This class does
  * not extend the Error class to prevent an invalid stack trace
@@ -11,12 +12,13 @@ exports.UncaughtError = void 0;
  * do not use the Error class (strings, etc) to an object.
  */
 class UncaughtError {
-  constructor(message, filename, lineno, colno) {
+  constructor(message, filename, lineno, colno, newrelic) {
     this.name = 'UncaughtError';
-    this.message = message;
+    this.message = typeof message === 'string' ? message : (0, _stringify.stringify)(message);
     this.sourceURL = filename;
     this.line = lineno;
     this.column = colno;
+    this.__newrelic = newrelic;
   }
 }
 exports.UncaughtError = UncaughtError;

package/dist/cjs/features/session_trace/aggregate/index.js

@@ -59,6 +59,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
     this.trace = {};
     this.nodeCount = 0;
     this.sentTrace = null;
+    this.prevStoredEvents = new Set();
     this.harvestTimeSeconds = (0, _config.getConfigurationValue)(agentIdentifier, 'session_trace.harvestTimeSeconds') || 10;
     this.maxNodesPerHarvest = (0, _config.getConfigurationValue)(agentIdentifier, 'session_trace.maxNodesPerHarvest') || 1000;
     /**
@@ -126,7 +127,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
         return controlTraceOp(on);
       }, this.featureName, this.ee);
     } else {
-      (0, _registerHandler.registerHandler)('errorAgg', () => {
+      (0, _registerHandler.registerHandler)('trace-jserror', () => {
         seenAnError = true;
         switchToFull();
       }, this.featureName, this.ee);
@@ -221,7 +222,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
       }
       return operationalGate.settle(() => _this.storeSTN(...args));
     }, this.featureName, this.ee);
-    (0, _registerHandler.registerHandler)('errorAgg', function () {
+    (0, _registerHandler.registerHandler)('trace-jserror', function () {
       for (var _len6 = arguments.length, args = new Array(_len6), _key6 = 0; _key6 < _len6; _key6++) {
         args[_key6] = arguments[_key6];
       }
@@ -270,6 +271,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
     }
   }
   #prepareHarvest(options) {
+    this.prevStoredEvents.clear(); // release references to past events for GC
     if (this.isStandalone) {
       if (this.ptid && (0, _now.now)() >= MAX_TRACE_DURATION) {
         // Perform a final harvest once we hit or exceed the max session trace time
@@ -336,6 +338,8 @@ class Aggregate extends _aggregateBase.AggregateBase {
   // Tracks the events and their listener's duration on objects wrapped by wrap-events.
   storeEvent(currentEvent, target, start, end) {
     if (this.shouldIgnoreEvent(currentEvent, target)) return;
+    if (this.prevStoredEvents.has(currentEvent)) return; // prevent multiple listeners of an event from creating duplicate trace nodes per occurrence. Cleared every harvest. near-zero chance for re-duplication after clearing per harvest since the timestamps of the event are considered for uniqueness.
+    this.prevStoredEvents.add(currentEvent);
     const evt = {
       n: this.evtName(currentEvent.type),
       s: start,

package/dist/cjs/features/spa/aggregate/index.js

@@ -651,7 +651,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
         state.interactionsSent = [];
       }
     }
-    baseEE.on('errorAgg', function (type, name, params, metrics) {
+    baseEE.on('spa-jserror', function (type, name, params, metrics) {
       if (!state.currentNode) return;
       params._interactionId = state.currentNode.interaction.id;
       // do not capture parentNodeId when in root node
@@ -659,6 +659,16 @@ class Aggregate extends _aggregateBase.AggregateBase {
         params._interactionNodeId = state.currentNode.id;
       }
     });
+    (0, _registerHandler.registerHandler)('function-err', function (args, obj, error) {
+      if (!state.currentNode) return;
+      error.__newrelic ??= {};
+      error.__newrelic[agentIdentifier] = {
+        interactionId: state.currentNode.interaction.id
+      };
+      if (state.currentNode.type && state.currentNode.type !== 'interaction') {
+        error.__newrelic[agentIdentifier].interactionNodeId = state.currentNode.id;
+      }
+    }, this.featureName, baseEE);
     baseEE.on('interaction', saveInteraction);
     function getActionText(node) {
       var nodeType = node.tagName.toLowerCase();

package/dist/cjs/features/spa/instrument/index.js

@@ -10,6 +10,7 @@ var _instrumentBase = require("../../utils/instrument-base");
 var CONSTANTS = _interopRequireWildcard(require("../constants"));
 var _runtime = require("../../../common/constants/runtime");
 var _now = require("../../../common/timing/now");
+var _handle = require("../../../common/event-emitter/handle");
 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); }
 function _interopRequireWildcard(e, r) { if (!r && e && e.__esModule) return e; if (null === e || "object" != typeof e && "function" != typeof e) return { default: e }; var t = _getRequireWildcardCache(r); if (t && t.has(e)) return t.get(e); var n = { __proto__: null }, a = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var u in e) if ("default" !== u && Object.prototype.hasOwnProperty.call(e, u)) { var i = a ? Object.getOwnPropertyDescriptor(e, u) : null; i && (i.get || i.set) ? Object.defineProperty(n, u, i) : n[u] = e[u]; } return n.default = e, t && t.set(e, n), n; }
 /*
@@ -32,8 +33,10 @@ const {
 class Instrument extends _instrumentBase.InstrumentBase {
   static featureName = FEATURE_NAME;
   constructor(agentIdentifier, aggregator) {
+    var _this;
     let auto = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : true;
     super(agentIdentifier, aggregator, FEATURE_NAME, auto);
+    _this = this;
     if (!_runtime.isBrowserScope) return; // SPA not supported outside web env
 
     try {
@@ -56,6 +59,12 @@ class Instrument extends _instrumentBase.InstrumentBase {
     this.ee.on(FN_END, endTimestamp);
     promiseEE.on(CB_END, endTimestamp);
     jsonpEE.on(CB_END, endTimestamp);
+    this.ee.on('fn-err', function () {
+      for (var _len = arguments.length, args = new Array(_len), _key = 0; _key < _len; _key++) {
+        args[_key] = arguments[_key];
+      }
+      if (!args[2]?.__newrelic?.[agentIdentifier]) (0, _handle.handle)('function-err', [...args], undefined, _this.featureName, _this.ee);
+    });
     this.ee.buffer([FN_START, FN_END, 'xhr-resolved'], this.featureName);
     eventsEE.buffer([FN_START], this.featureName);
     timerEE.buffer(['setTimeout' + END, 'clearTimeout' + START, FN_START], this.featureName);

package/dist/cjs/features/utils/instrument-base.js

@@ -57,7 +57,7 @@ class InstrumentBase extends _featureBase.FeatureBase {
     if ((0, _config.getConfigurationValue)(this.agentIdentifier, "".concat(this.featureName, ".autoStart")) === false) this.auto = false;
     /** if the feature requires opt-in (!auto-start), it will get registered once the api has been called */
     if (this.auto) (0, _drain.registerDrain)(agentIdentifier, featureName);else {
-      this.ee.on("".concat(this.featureName, "-opt-in"), (0, _invoke.single)(() => {
+      this.ee.on('manual-start-all', (0, _invoke.single)(() => {
         // register the feature to drain only once the API has been called, it will drain when importAggregator finishes for all the features
         // called by the api in that cycle
         (0, _drain.registerDrain)(this.agentIdentifier, this.featureName);

package/dist/cjs/loaders/api/api.js

@@ -133,19 +133,10 @@ function setAPI(agentIdentifier, forceDrain) {
     }
     return appendJsAttribute('application.version', value, 'setApplicationVersion', false);
   };
-  apiInterface.start = features => {
+  apiInterface.start = () => {
     try {
-      const smTag = !features ? 'undefined' : 'defined';
-      (0, _handle.handle)(_constants.SUPPORTABILITY_METRIC_CHANNEL, ["API/start/".concat(smTag, "/called")], undefined, _features.FEATURE_NAMES.metrics, instanceEE);
-      const featNames = Object.values(_features.FEATURE_NAMES);
-      if (features === undefined) features = featNames;else {
-        features = Array.isArray(features) && features.length ? features : [features];
-        if (features.some(f => !featNames.includes(f))) return (0, _console.warn)("Invalid feature name supplied. Acceptable feature names are: ".concat(featNames));
-        if (!features.includes(_features.FEATURE_NAMES.pageViewEvent)) features.push(_features.FEATURE_NAMES.pageViewEvent);
-      }
-      features.forEach(feature => {
-        instanceEE.emit("".concat(feature, "-opt-in"));
-      });
+      (0, _handle.handle)(_constants.SUPPORTABILITY_METRIC_CHANNEL, ['API/start/called'], undefined, _features.FEATURE_NAMES.metrics, instanceEE);
+      instanceEE.emit('manual-start-all');
     } catch (err) {
       (0, _console.warn)('An unexpected issue occurred', err);
     }
@@ -176,9 +167,10 @@ function setAPI(agentIdentifier, forceDrain) {
           try {
             return cb.apply(this, arguments);
           } catch (err) {
-            tracerEE.emit('fn-err', [arguments, this, err], contextStore);
+            const error = typeof err === 'string' ? new Error(err) : err;
+            tracerEE.emit('fn-err', [arguments, this, error], contextStore);
             // the error came from outside the agent, so don't swallow
-            throw err;
+            throw error;
           } finally {
             tracerEE.emit('fn-end', [(0, _now.now)()], contextStore);
           }

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

@@ -6,7 +6,7 @@
 /**
  * Exposes the version of the agent
  */
-export const VERSION = "1.258.0";
+export const VERSION = "1.258.1";
 
 /**
  * Exposes the build type of the agent

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

@@ -6,7 +6,7 @@
 /**
  * Exposes the version of the agent
  */
-export const VERSION = "1.258.0";
+export const VERSION = "1.258.1";
 
 /**
  * Exposes the build type of the agent

package/dist/esm/common/deny-list/deny-list.js

@@ -10,14 +10,8 @@ var denyList = [];
  * @returns {boolean} `true` if request does not match any entries of {@link denyList|deny list}; else `false`
  */
 export function shouldCollectEvent(params) {
-  if (denyList.length === 0) {
-    return true;
-  }
-
-  // XHR requests with an undefined hostname (e.g., data URLs) should not be collected.
-  if (params.hostname === undefined) {
-    return false;
-  }
+  if (hasUndefinedHostname(params)) return false;
+  if (denyList.length === 0) return true;
   for (var i = 0; i < denyList.length; i++) {
     var parsed = denyList[i];
     if (parsed.hostname === '*') {
@@ -29,6 +23,9 @@ export function shouldCollectEvent(params) {
   }
   return true;
 }
+export function hasUndefinedHostname(params) {
+  return params.hostname === undefined; // requests with an undefined hostname (e.g., data URLs) should not be collected.
+}
 
 /**
  * Initializes the {@link denyList|XHR deny list} by extracting hostname and pathname from an array of filter strings.

package/dist/esm/common/vitals/time-to-first-byte.js

@@ -3,7 +3,15 @@ import { VITAL_NAMES } from './constants';
 import { VitalMetric } from './vital-metric';
 import { onTTFB } from 'web-vitals/attribution';
 export const timeToFirstByte = new VitalMetric(VITAL_NAMES.TIME_TO_FIRST_BYTE);
-if (isBrowserScope && typeof PerformanceNavigationTiming !== 'undefined' && !isiOS) {
+
+/**
+ * onTTFB is not supported in the following scenarios:
+ * - in a non-browser scope
+ * - in browsers that do not support PerformanceNavigationTiming API
+ * - in an iOS browser
+ * - cross-origin iframes specifically in firefox and safari
+ */
+if (isBrowserScope && typeof PerformanceNavigationTiming !== 'undefined' && !isiOS && window === window.parent) {
   onTTFB(_ref => {
     let {
       value,