@newrelic/browser-agent 1.241.0 → 1.242.0

package/dist/cjs/cdn/polyfills/lite.js

@@ -1,4 +1,16 @@
 "use strict";
 
 require("../polyfills.js");
-require("../lite");
+var _agent = require("../../loaders/agent");
+var _instrument = require("../../features/page_view_event/instrument");
+var _instrument2 = require("../../features/page_view_timing/instrument");
+var _instrument3 = require("../../features/metrics/instrument");
+/**
+ * @file Creates a version of the "Lite" agent loader with [core-js]{@link https://github.com/zloirock/core-js}
+ * polyfills for pre-ES6 browsers and IE 11.
+ */
+
+new _agent.Agent({
+  features: [_instrument.Instrument, _instrument2.Instrument, _instrument3.Instrument],
+  loaderType: 'lite-polyfills'
+});

package/dist/cjs/cdn/polyfills/pro.js

@@ -1,4 +1,20 @@
 "use strict";
 
 require("../polyfills.js");
-require("../pro");
+var _agent = require("../../loaders/agent");
+var _instrument = require("../../features/page_view_event/instrument");
+var _instrument2 = require("../../features/page_view_timing/instrument");
+var _instrument3 = require("../../features/metrics/instrument");
+var _instrument4 = require("../../features/jserrors/instrument");
+var _instrument5 = require("../../features/ajax/instrument");
+var _instrument6 = require("../../features/session_trace/instrument");
+var _instrument7 = require("../../features/page_action/instrument");
+/**
+ * @file Creates a version of the "PRO" agent loader with [core-js]{@link https://github.com/zloirock/core-js}
+ * polyfills for pre-ES6 browsers and IE 11.
+ */
+
+new _agent.Agent({
+  features: [_instrument.Instrument, _instrument2.Instrument, _instrument6.Instrument, _instrument5.Instrument, _instrument3.Instrument, _instrument7.Instrument, _instrument4.Instrument],
+  loaderType: 'pro-polyfills'
+});

package/dist/cjs/cdn/polyfills/spa.js

@@ -1,4 +1,21 @@
 "use strict";
 
 require("../polyfills.js");
-require("../spa");
+var _agent = require("../../loaders/agent");
+var _instrument = require("../../features/page_view_event/instrument");
+var _instrument2 = require("../../features/page_view_timing/instrument");
+var _instrument3 = require("../../features/metrics/instrument");
+var _instrument4 = require("../../features/jserrors/instrument");
+var _instrument5 = require("../../features/ajax/instrument");
+var _instrument6 = require("../../features/session_trace/instrument");
+var _instrument7 = require("../../features/spa/instrument");
+var _instrument8 = require("../../features/page_action/instrument");
+/**
+ * @file Creates a version of the "SPA" agent loader with [core-js]{@link https://github.com/zloirock/core-js}
+ * polyfills for pre-ES6 browsers and IE 11.
+ */
+
+new _agent.Agent({
+  features: [_instrument5.Instrument, _instrument.Instrument, _instrument2.Instrument, _instrument6.Instrument, _instrument3.Instrument, _instrument8.Instrument, _instrument4.Instrument, _instrument7.Instrument],
+  loaderType: 'spa-polyfills'
+});

package/dist/cjs/common/config/state/init.js

@@ -6,16 +6,36 @@ Object.defineProperty(exports, "__esModule", {
 exports.getConfiguration = getConfiguration;
 exports.getConfigurationValue = getConfigurationValue;
 exports.setConfiguration = setConfiguration;
+var _querySelector = require("../../dom/query-selector");
 var _constants = require("../../session/constants");
+var _console = require("../../util/console");
 var _nreum = require("../../window/nreum");
 var _configurable = require("./configurable");
 const model = () => {
   const hiddenState = {
+    mask_selector: '*',
     block_selector: '[data-nr-block]',
     mask_input_options: {
-      password: true
+      color: false,
+      date: false,
+      'datetime-local': false,
+      email: false,
+      month: false,
+      number: false,
+      range: false,
+      search: false,
+      tel: false,
+      text: false,
+      time: false,
+      url: false,
+      week: false,
+      // unify textarea and select element with text input
+      textarea: false,
+      select: false,
+      password: true // This will be enforced to always be true in the setter
     }
   };
+
   return {
     proxy: {
       assets: undefined,
@@ -91,8 +111,15 @@ const model = () => {
       error_sampling_rate: 50,
       // float from 0 - 100
       // recording config settings
-      mask_text_selector: '*',
       mask_all_inputs: true,
+      // this has a getter/setter to facilitate validation of the selectors
+      get mask_text_selector() {
+        return hiddenState.mask_selector;
+      },
+      set mask_text_selector(val) {
+        if ((0, _querySelector.isValidSelector)(val)) hiddenState.mask_selector = val + ',[data-nr-mask]';else if (val === null) hiddenState.mask_selector = val; // null is acceptable, which completely disables the behavior
+        else (0, _console.warn)('An invalid session_replay.mask_selector was provided and will not be used', val);
+      },
       // these properties only have getters because they are enforcable constants and should error if someone tries to override them
       get block_class() {
         return 'nr-block';
@@ -109,17 +136,17 @@ const model = () => {
         return hiddenState.block_selector;
       },
       set block_selector(val) {
-        hiddenState.block_selector += ",".concat(val);
+        if ((0, _querySelector.isValidSelector)(val)) hiddenState.block_selector += ",".concat(val);else if (val !== '') (0, _console.warn)('An invalid session_replay.block_selector was provided and will not be used', val);
       },
       // password: must always be present and true no matter what customer sets
       get mask_input_options() {
         return hiddenState.mask_input_options;
       },
       set mask_input_options(val) {
-        hiddenState.mask_input_options = {
+        if (val && typeof val === 'object') hiddenState.mask_input_options = {
           ...val,
           password: true
-        };
+        };else (0, _console.warn)('An invalid session_replay.mask_input_option was provided and will not be used', val);
       }
     },
     spa: {

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

@@ -12,7 +12,7 @@ exports.VERSION = exports.DIST_METHOD = exports.BUILD_ENV = void 0;
 /**
  * Exposes the version of the agent
  */
-const VERSION = "1.241.0";
+const VERSION = "1.242.0";
 
 /**
  * Exposes the build type of the agent

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

@@ -12,7 +12,7 @@ exports.VERSION = exports.DIST_METHOD = exports.BUILD_ENV = void 0;
 /**
  * Exposes the version of the agent
  */
-const VERSION = "1.241.0";
+const VERSION = "1.242.0";
 
 /**
  * Exposes the build type of the agent

package/dist/cjs/common/dom/query-selector.js

@@ -0,0 +1,16 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.isValidSelector = void 0;
+const isValidSelector = selector => {
+  if (!selector || typeof selector !== 'string') return false;
+  try {
+    document.createDocumentFragment().querySelector(selector);
+  } catch {
+    return false;
+  }
+  return true;
+};
+exports.isValidSelector = isValidSelector;

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

@@ -15,6 +15,8 @@ var _sharedChannel = require("../../../common/constants/shared-channel");
 var _encode = require("../../../common/url/encode");
 var _console = require("../../../common/util/console");
 var _runtime = require("../../../common/constants/runtime");
+var _constants2 = require("../../metrics/constants");
+var _features = require("../../../loaders/features/features");
 function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "function") return null; var cacheBabelInterop = new WeakMap(); var cacheNodeInterop = new WeakMap(); return (_getRequireWildcardCache = function (nodeInterop) { return nodeInterop ? cacheNodeInterop : cacheBabelInterop; })(nodeInterop); }
 function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || typeof obj !== "object" && typeof obj !== "function") { return { default: obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj.default = obj; if (cache) { cache.set(obj, newObj); } return newObj; } /*
                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                       * Copyright 2023 New Relic Corporation. All rights reserved.
@@ -104,7 +106,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
     if (shouldSetup) {
       // The SessionEntity class can emit a message indicating the session was cleared and reset (expiry, inactivity). This feature must abort and never resume if that occurs.
       this.ee.on(_sessionEntity.SESSION_EVENTS.RESET, () => {
-        this.abort();
+        this.abort('Session Reset');
       });
 
       // The SessionEntity class can emit a message indicating the session was paused (visibility change). This feature must stop recording if that occurs.
@@ -191,7 +193,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
       // Do not change the webpackChunkName or it will break the webpack nrba-chunking plugin
       recorder = (await Promise.resolve().then(() => _interopRequireWildcard(require( /* webpackChunkName: "recorder" */'rrweb')))).record;
     } catch (err) {
-      return this.abort();
+      return this.abort('Recorder failed to import');
     }
 
     // FULL mode records AND reports from the beginning, while ERROR mode only records (but does not report).
@@ -267,7 +269,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
   onHarvestFinished(result) {
     // The mutual decision for now is to stop recording and clear buffers if ingest is experiencing 429 rate limiting
     if (result.status === 429) {
-      this.abort();
+      this.abort('429: Too many requests');
     }
     if (this.blocked) this.scheduler.stopTimer(true);
   }
@@ -287,7 +289,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
   startRecording() {
     if (!recorder) {
       (0, _console.warn)('Recording library was never imported');
-      return this.abort();
+      return this.abort('Recorder was never imported');
     }
     this.clearTimestamps();
     // set the fallbacks as early as possible
@@ -331,7 +333,8 @@ class Aggregate extends _aggregateBase.AggregateBase {
     // Vortex will block payloads at a certain size, we might as well not send.
     if (payloadSize > MAX_PAYLOAD_SIZE) {
       this.clearBuffer();
-      return this.abort();
+      this.ee.emit(_constants2.SUPPORTABILITY_METRIC_CHANNEL, ['SessionReplay/Too-Big/Seen'], undefined, _features.FEATURE_NAMES.metrics, this.ee);
+      return this.abort('Payload too big');
     }
     // Checkout events are flags by the recording lib that indicate a fullsnapshot was taken every n ms. These are important
     // to help reconstruct the replay later and must be included.  While waiting and buffering for errors to come through,
@@ -402,7 +405,8 @@ class Aggregate extends _aggregateBase.AggregateBase {
   }
 
   /** Abort the feature, once aborted it will not resume */
-  abort() {
+  abort(reason) {
+    (0, _console.warn)("SR aborted -- ".concat(reason));
     this.blocked = true;
     this.mode = _sessionEntity.MODE.OFF;
     this.stopRecording();

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

@@ -119,6 +119,7 @@ class InstrumentBase extends _featureBase.FeatureBase {
         (0, _console.warn)("Downloading and initializing ".concat(this.featureName, " failed..."), e);
         this.abortHandler?.(); // undo any important alterations made to the page
         // not supported yet but nice to do: "abort" this agent's EE for this feature specifically
+        (0, _drain.drain)(this.agentIdentifier, this.featureName);
         loadedSuccessfully(false);
       }
     };

package/dist/esm/cdn/polyfills/lite.js

@@ -4,4 +4,11 @@
  */
 
 import '../polyfills.js';
-import '../lite';
+import { Agent } from '../../loaders/agent';
+import { Instrument as InstrumentPageViewEvent } from '../../features/page_view_event/instrument';
+import { Instrument as InstrumentPageViewTiming } from '../../features/page_view_timing/instrument';
+import { Instrument as InstrumentMetrics } from '../../features/metrics/instrument';
+new Agent({
+  features: [InstrumentPageViewEvent, InstrumentPageViewTiming, InstrumentMetrics],
+  loaderType: 'lite-polyfills'
+});

package/dist/esm/cdn/polyfills/pro.js

@@ -1,7 +1,18 @@
 /**
- * @file Creates a version of the "Pro" agent loader with [core-js]{@link https://github.com/zloirock/core-js}
+ * @file Creates a version of the "PRO" agent loader with [core-js]{@link https://github.com/zloirock/core-js}
  * polyfills for pre-ES6 browsers and IE 11.
  */
 
 import '../polyfills.js';
-import '../pro';
+import { Agent } from '../../loaders/agent';
+import { Instrument as InstrumentPageViewEvent } from '../../features/page_view_event/instrument';
+import { Instrument as InstrumentPageViewTiming } from '../../features/page_view_timing/instrument';
+import { Instrument as InstrumentMetrics } from '../../features/metrics/instrument';
+import { Instrument as InstrumentErrors } from '../../features/jserrors/instrument';
+import { Instrument as InstrumentXhr } from '../../features/ajax/instrument';
+import { Instrument as InstrumentSessionTrace } from '../../features/session_trace/instrument';
+import { Instrument as InstrumentPageAction } from '../../features/page_action/instrument';
+new Agent({
+  features: [InstrumentPageViewEvent, InstrumentPageViewTiming, InstrumentSessionTrace, InstrumentXhr, InstrumentMetrics, InstrumentPageAction, InstrumentErrors],
+  loaderType: 'pro-polyfills'
+});

package/dist/esm/cdn/polyfills/spa.js

@@ -4,4 +4,16 @@
  */
 
 import '../polyfills.js';
-import '../spa';
+import { Agent } from '../../loaders/agent';
+import { Instrument as InstrumentPageViewEvent } from '../../features/page_view_event/instrument';
+import { Instrument as InstrumentPageViewTiming } from '../../features/page_view_timing/instrument';
+import { Instrument as InstrumentMetrics } from '../../features/metrics/instrument';
+import { Instrument as InstrumentErrors } from '../../features/jserrors/instrument';
+import { Instrument as InstrumentXhr } from '../../features/ajax/instrument';
+import { Instrument as InstrumentSessionTrace } from '../../features/session_trace/instrument';
+import { Instrument as InstrumentSpa } from '../../features/spa/instrument';
+import { Instrument as InstrumentPageAction } from '../../features/page_action/instrument';
+new Agent({
+  features: [InstrumentXhr, InstrumentPageViewEvent, InstrumentPageViewTiming, InstrumentSessionTrace, InstrumentMetrics, InstrumentPageAction, InstrumentErrors, InstrumentSpa],
+  loaderType: 'spa-polyfills'
+});

package/dist/esm/common/config/state/init.js

@@ -1,13 +1,33 @@
+import { isValidSelector } from '../../dom/query-selector';
 import { DEFAULT_EXPIRES_MS, DEFAULT_INACTIVE_MS } from '../../session/constants';
+import { warn } from '../../util/console';
 import { gosNREUMInitializedAgents } from '../../window/nreum';
 import { getModeledObject } from './configurable';
 const model = () => {
   const hiddenState = {
+    mask_selector: '*',
     block_selector: '[data-nr-block]',
     mask_input_options: {
-      password: true
+      color: false,
+      date: false,
+      'datetime-local': false,
+      email: false,
+      month: false,
+      number: false,
+      range: false,
+      search: false,
+      tel: false,
+      text: false,
+      time: false,
+      url: false,
+      week: false,
+      // unify textarea and select element with text input
+      textarea: false,
+      select: false,
+      password: true // This will be enforced to always be true in the setter
     }
   };
+
   return {
     proxy: {
       assets: undefined,
@@ -83,8 +103,15 @@ const model = () => {
       error_sampling_rate: 50,
       // float from 0 - 100
       // recording config settings
-      mask_text_selector: '*',
       mask_all_inputs: true,
+      // this has a getter/setter to facilitate validation of the selectors
+      get mask_text_selector() {
+        return hiddenState.mask_selector;
+      },
+      set mask_text_selector(val) {
+        if (isValidSelector(val)) hiddenState.mask_selector = val + ',[data-nr-mask]';else if (val === null) hiddenState.mask_selector = val; // null is acceptable, which completely disables the behavior
+        else warn('An invalid session_replay.mask_selector was provided and will not be used', val);
+      },
       // these properties only have getters because they are enforcable constants and should error if someone tries to override them
       get block_class() {
         return 'nr-block';
@@ -101,17 +128,17 @@ const model = () => {
         return hiddenState.block_selector;
       },
       set block_selector(val) {
-        hiddenState.block_selector += ",".concat(val);
+        if (isValidSelector(val)) hiddenState.block_selector += ",".concat(val);else if (val !== '') warn('An invalid session_replay.block_selector was provided and will not be used', val);
       },
       // password: must always be present and true no matter what customer sets
       get mask_input_options() {
         return hiddenState.mask_input_options;
       },
       set mask_input_options(val) {
-        hiddenState.mask_input_options = {
+        if (val && typeof val === 'object') hiddenState.mask_input_options = {
           ...val,
           password: true
-        };
+        };else warn('An invalid session_replay.mask_input_option was provided and will not be used', val);
       }
     },
     spa: {

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

@@ -6,7 +6,7 @@
 /**
  * Exposes the version of the agent
  */
-export const VERSION = "1.241.0";
+export const VERSION = "1.242.0";
 
 /**
  * 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.241.0";
+export const VERSION = "1.242.0";
 
 /**
  * Exposes the build type of the agent

package/dist/esm/common/dom/query-selector.js

@@ -0,0 +1,9 @@
+export const isValidSelector = selector => {
+  if (!selector || typeof selector !== 'string') return false;
+  try {
+    document.createDocumentFragment().querySelector(selector);
+  } catch {
+    return false;
+  }
+  return true;
+};

package/dist/esm/features/session_replay/aggregate/index.js

@@ -21,6 +21,8 @@ import { sharedChannel } from '../../../common/constants/shared-channel';
 import { obj as encodeObj } from '../../../common/url/encode';
 import { warn } from '../../../common/util/console';
 import { globalScope } from '../../../common/constants/runtime';
+import { SUPPORTABILITY_METRIC_CHANNEL } from '../../metrics/constants';
+import { FEATURE_NAMES } from '../../../loaders/features/features';
 
 // would be better to get this dynamically in some way
 export const RRWEB_VERSION = '2.0.0-alpha.8';
@@ -96,7 +98,7 @@ export class Aggregate extends AggregateBase {
     if (shouldSetup) {
       // The SessionEntity class can emit a message indicating the session was cleared and reset (expiry, inactivity). This feature must abort and never resume if that occurs.
       this.ee.on(SESSION_EVENTS.RESET, () => {
-        this.abort();
+        this.abort('Session Reset');
       });
 
       // The SessionEntity class can emit a message indicating the session was paused (visibility change). This feature must stop recording if that occurs.
@@ -183,7 +185,7 @@ export class Aggregate extends AggregateBase {
       // Do not change the webpackChunkName or it will break the webpack nrba-chunking plugin
       recorder = (await import( /* webpackChunkName: "recorder" */'rrweb')).record;
     } catch (err) {
-      return this.abort();
+      return this.abort('Recorder failed to import');
     }
 
     // FULL mode records AND reports from the beginning, while ERROR mode only records (but does not report).
@@ -259,7 +261,7 @@ export class Aggregate extends AggregateBase {
   onHarvestFinished(result) {
     // The mutual decision for now is to stop recording and clear buffers if ingest is experiencing 429 rate limiting
     if (result.status === 429) {
-      this.abort();
+      this.abort('429: Too many requests');
     }
     if (this.blocked) this.scheduler.stopTimer(true);
   }
@@ -279,7 +281,7 @@ export class Aggregate extends AggregateBase {
   startRecording() {
     if (!recorder) {
       warn('Recording library was never imported');
-      return this.abort();
+      return this.abort('Recorder was never imported');
     }
     this.clearTimestamps();
     // set the fallbacks as early as possible
@@ -323,7 +325,8 @@ export class Aggregate extends AggregateBase {
     // Vortex will block payloads at a certain size, we might as well not send.
     if (payloadSize > MAX_PAYLOAD_SIZE) {
       this.clearBuffer();
-      return this.abort();
+      this.ee.emit(SUPPORTABILITY_METRIC_CHANNEL, ['SessionReplay/Too-Big/Seen'], undefined, FEATURE_NAMES.metrics, this.ee);
+      return this.abort('Payload too big');
     }
     // Checkout events are flags by the recording lib that indicate a fullsnapshot was taken every n ms. These are important
     // to help reconstruct the replay later and must be included.  While waiting and buffering for errors to come through,
@@ -394,7 +397,8 @@ export class Aggregate extends AggregateBase {
   }
 
   /** Abort the feature, once aborted it will not resume */
-  abort() {
+  abort(reason) {
+    warn("SR aborted -- ".concat(reason));
     this.blocked = true;
     this.mode = MODE.OFF;
     this.stopRecording();

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

@@ -114,6 +114,7 @@ export class InstrumentBase extends FeatureBase {
         warn("Downloading and initializing ".concat(this.featureName, " failed..."), e);
         this.abortHandler?.(); // undo any important alterations made to the page
         // not supported yet but nice to do: "abort" this agent's EE for this feature specifically
+        drain(this.agentIdentifier, this.featureName);
         loadedSuccessfully(false);
       }
     };

package/dist/types/common/config/state/init.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"init.d.ts","sourceRoot":"","sources":["../../../../../src/common/config/state/init.js"],"names":[],"mappings":"AA0EA,+CAIC;AAED,0DAIC;AAED,+DAYC"}
+{"version":3,"file":"init.d.ts","sourceRoot":"","sources":["../../../../../src/common/config/state/init.js"],"names":[],"mappings":"AAuGA,+CAIC;AAED,0DAIC;AAED,+DAYC"}

package/dist/types/common/dom/query-selector.d.ts

@@ -0,0 +1,2 @@
+export function isValidSelector(selector: any): boolean;
+//# sourceMappingURL=query-selector.d.ts.map

package/dist/types/common/dom/query-selector.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"query-selector.d.ts","sourceRoot":"","sources":["../../../../src/common/dom/query-selector.js"],"names":[],"mappings":"AAAO,wDAQN"}

package/dist/types/features/session_replay/aggregate/index.d.ts

@@ -94,7 +94,7 @@ export class Aggregate extends AggregateBase {
     /** Estimate the payload size */
     getPayloadSize(newBytes?: number): any;
     /** Abort the feature, once aborted it will not resume */
-    abort(): void;
+    abort(reason: any): void;
     /** Extensive research has yielded about an 88% compression factor on these payloads.
      * This is an estimation using that factor as to not cause performance issues while evaluating
      * https://staging.onenr.io/037jbJWxbjy

package/dist/types/features/session_replay/aggregate/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../src/features/session_replay/aggregate/index.js"],"names":[],"mappings":"AAyBA,4CAA4C;AAE5C,mCAAmC;AAInC,uCAAuC;AACvC,uCAAuC;AACvC,iCAAiC;AACjC,uCAAuC;AAIvC;IACE,2BAAiC;IACjC,mDAmGC;IAjGC,iHAAiH;IACjH,cAAgB;IAChB,8GAA8G;IAC9G,wBAAgH;IAChH,iFAAiF;IACjF,qBAAwB;IACxB,mEAAmE;IACnE,sBAAyB;IACzB,6GAA6G;IAC7G,aAAoB;IAGpB,iEAAiE;IACjE,mBAAsB;IACtB,gDAAgD;IAChD,wBAA0B;IAE1B,gGAAgG;IAChG,sBAAyB;IACzB;;;MAGE;IACF,qBAAwB;IACxB,4IAA4I;IAC5I,iBAAoB;IACpB,+HAA+H;IAC/H,kBAAqB;IAErB;;OAEG;IACH;;;;;;;;;MAA+G;IAE/G,qGAAqG;IACrG,+BAA+B;IAE/B,kIAAkI;IAClI,cAAyB;IAOzB,uIAAuI;IACvI,0BAAyE;IAiBvE,wCAKQ;IA+BZ;;;;;;OAMG;IACH,kCALW,OAAO,eACP,OAAO,cACP,OAAO,GACL,IAAI,CAyDhB;IAED;;;;;;;;;oBAaC;IAED;;;;;;;;;MA4BC;IAED,qCAOC;IAED,kFAAkF;IAClF,oBAQC;IAED,qDAAqD;IACrD,uBA4BC;IAED,yHAAyH;IACzH,yCA4CC;IAED,0HAA0H;IAC1H,yBAGC;IAED,gCAQC;IAED,wBAEC;IAED,gCAAgC;IAChC,uCAGC;IAED,yDAAyD;IACzD,cAOC;IAED;;;SAGK;IACL,oCAGC;IAED,yCAGC;CACF;8BA3X6B,4BAA4B;iCALzB,2CAA2C"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../src/features/session_replay/aggregate/index.js"],"names":[],"mappings":"AA2BA,4CAA4C;AAE5C,mCAAmC;AAInC,uCAAuC;AACvC,uCAAuC;AACvC,iCAAiC;AACjC,uCAAuC;AAIvC;IACE,2BAAiC;IACjC,mDAmGC;IAjGC,iHAAiH;IACjH,cAAgB;IAChB,8GAA8G;IAC9G,wBAAgH;IAChH,iFAAiF;IACjF,qBAAwB;IACxB,mEAAmE;IACnE,sBAAyB;IACzB,6GAA6G;IAC7G,aAAoB;IAGpB,iEAAiE;IACjE,mBAAsB;IACtB,gDAAgD;IAChD,wBAA0B;IAE1B,gGAAgG;IAChG,sBAAyB;IACzB;;;MAGE;IACF,qBAAwB;IACxB,4IAA4I;IAC5I,iBAAoB;IACpB,+HAA+H;IAC/H,kBAAqB;IAErB;;OAEG;IACH;;;;;;;;;MAA+G;IAE/G,qGAAqG;IACrG,+BAA+B;IAE/B,kIAAkI;IAClI,cAAyB;IAOzB,uIAAuI;IACvI,0BAAyE;IAiBvE,wCAKQ;IA+BZ;;;;;;OAMG;IACH,kCALW,OAAO,eACP,OAAO,cACP,OAAO,GACL,IAAI,CAyDhB;IAED;;;;;;;;;oBAaC;IAED;;;;;;;;;MA4BC;IAED,qCAOC;IAED,kFAAkF;IAClF,oBAQC;IAED,qDAAqD;IACrD,uBA4BC;IAED,yHAAyH;IACzH,yCA6CC;IAED,0HAA0H;IAC1H,yBAGC;IAED,gCAQC;IAED,wBAEC;IAED,gCAAgC;IAChC,uCAGC;IAED,yDAAyD;IACzD,yBAQC;IAED;;;SAGK;IACL,oCAGC;IAED,yCAGC;CACF;8BA/X6B,4BAA4B;iCALzB,2CAA2C"}

package/dist/types/features/utils/instrument-base.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"instrument-base.d.ts","sourceRoot":"","sources":["../../../../src/features/utils/instrument-base.js"],"names":[],"mappings":"AAcA;;;GAGG;AACH;IACE;;;;;;;;OAQG;IACH,6BAPW,MAAM,uCAEN,MAAM,8BA4BhB;IArBC,cAAgB;IAEhB,8IAA8I;IAC9I,cADW,WAAW,SAAS,CACF;IAE7B;;;MAGE;IACF,qBAA8B;IAE9B;;;MAGE;IACF,kCAAoC;IAQtC;;;;;OAKG;IACH,mEA2DC;IAED;;;;;KAKC;IACD,6BAJS,MAAM,mCAWd;CACF;4BA/H2B,gBAAgB"}
+{"version":3,"file":"instrument-base.d.ts","sourceRoot":"","sources":["../../../../src/features/utils/instrument-base.js"],"names":[],"mappings":"AAcA;;;GAGG;AACH;IACE;;;;;;;;OAQG;IACH,6BAPW,MAAM,uCAEN,MAAM,8BA4BhB;IArBC,cAAgB;IAEhB,8IAA8I;IAC9I,cADW,WAAW,SAAS,CACF;IAE7B;;;MAGE;IACF,qBAA8B;IAE9B;;;MAGE;IACF,kCAAoC;IAQtC;;;;;OAKG;IACH,mEA4DC;IAED;;;;;KAKC;IACD,6BAJS,MAAM,mCAWd;CACF;4BAhI2B,gBAAgB"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@newrelic/browser-agent",
-  "version": "1.241.0",
+  "version": "1.242.0",
   "private": false,
   "author": "New Relic Browser Agent Team <browser-agent@newrelic.com>",
   "description": "New Relic Browser Agent",

package/src/cdn/polyfills/lite.js

@@ -4,4 +4,17 @@
  */
 
 import '../polyfills.js'
-import '../lite'
+import { Agent } from '../../loaders/agent'
+
+import { Instrument as InstrumentPageViewEvent } from '../../features/page_view_event/instrument'
+import { Instrument as InstrumentPageViewTiming } from '../../features/page_view_timing/instrument'
+import { Instrument as InstrumentMetrics } from '../../features/metrics/instrument'
+
+new Agent({
+  features: [
+    InstrumentPageViewEvent,
+    InstrumentPageViewTiming,
+    InstrumentMetrics
+  ],
+  loaderType: 'lite-polyfills'
+})

package/src/cdn/polyfills/pro.js

@@ -1,7 +1,28 @@
 /**
- * @file Creates a version of the "Pro" agent loader with [core-js]{@link https://github.com/zloirock/core-js}
+ * @file Creates a version of the "PRO" agent loader with [core-js]{@link https://github.com/zloirock/core-js}
  * polyfills for pre-ES6 browsers and IE 11.
  */
 
 import '../polyfills.js'
-import '../pro'
+import { Agent } from '../../loaders/agent'
+
+import { Instrument as InstrumentPageViewEvent } from '../../features/page_view_event/instrument'
+import { Instrument as InstrumentPageViewTiming } from '../../features/page_view_timing/instrument'
+import { Instrument as InstrumentMetrics } from '../../features/metrics/instrument'
+import { Instrument as InstrumentErrors } from '../../features/jserrors/instrument'
+import { Instrument as InstrumentXhr } from '../../features/ajax/instrument'
+import { Instrument as InstrumentSessionTrace } from '../../features/session_trace/instrument'
+import { Instrument as InstrumentPageAction } from '../../features/page_action/instrument'
+
+new Agent({
+  features: [
+    InstrumentPageViewEvent,
+    InstrumentPageViewTiming,
+    InstrumentSessionTrace,
+    InstrumentXhr,
+    InstrumentMetrics,
+    InstrumentPageAction,
+    InstrumentErrors
+  ],
+  loaderType: 'pro-polyfills'
+})

package/src/cdn/polyfills/spa.js

@@ -4,4 +4,27 @@
  */
 
 import '../polyfills.js'
-import '../spa'
+import { Agent } from '../../loaders/agent'
+
+import { Instrument as InstrumentPageViewEvent } from '../../features/page_view_event/instrument'
+import { Instrument as InstrumentPageViewTiming } from '../../features/page_view_timing/instrument'
+import { Instrument as InstrumentMetrics } from '../../features/metrics/instrument'
+import { Instrument as InstrumentErrors } from '../../features/jserrors/instrument'
+import { Instrument as InstrumentXhr } from '../../features/ajax/instrument'
+import { Instrument as InstrumentSessionTrace } from '../../features/session_trace/instrument'
+import { Instrument as InstrumentSpa } from '../../features/spa/instrument'
+import { Instrument as InstrumentPageAction } from '../../features/page_action/instrument'
+
+new Agent({
+  features: [
+    InstrumentXhr,
+    InstrumentPageViewEvent,
+    InstrumentPageViewTiming,
+    InstrumentSessionTrace,
+    InstrumentMetrics,
+    InstrumentPageAction,
+    InstrumentErrors,
+    InstrumentSpa
+  ],
+  loaderType: 'spa-polyfills'
+})

package/src/common/config/state/init.js

@@ -1,11 +1,32 @@
+import { isValidSelector } from '../../dom/query-selector'
 import { DEFAULT_EXPIRES_MS, DEFAULT_INACTIVE_MS } from '../../session/constants'
+import { warn } from '../../util/console'
 import { gosNREUMInitializedAgents } from '../../window/nreum'
 import { getModeledObject } from './configurable'
 
 const model = () => {
   const hiddenState = {
+    mask_selector: '*',
     block_selector: '[data-nr-block]',
-    mask_input_options: { password: true }
+    mask_input_options: {
+      color: false,
+      date: false,
+      'datetime-local': false,
+      email: false,
+      month: false,
+      number: false,
+      range: false,
+      search: false,
+      tel: false,
+      text: false,
+      time: false,
+      url: false,
+      week: false,
+      // unify textarea and select element with text input
+      textarea: false,
+      select: false,
+      password: true // This will be enforced to always be true in the setter
+    }
   }
   return {
     proxy: {
@@ -43,8 +64,14 @@ const model = () => {
       sampling_rate: 50, // float from 0 - 100
       error_sampling_rate: 50, // float from 0 - 100
       // recording config settings
-      mask_text_selector: '*',
       mask_all_inputs: true,
+      // this has a getter/setter to facilitate validation of the selectors
+      get mask_text_selector () { return hiddenState.mask_selector },
+      set mask_text_selector (val) {
+        if (isValidSelector(val)) hiddenState.mask_selector = val + ',[data-nr-mask]'
+        else if (val === null) hiddenState.mask_selector = val // null is acceptable, which completely disables the behavior
+        else warn('An invalid session_replay.mask_selector was provided and will not be used', val)
+      },
       // these properties only have getters because they are enforcable constants and should error if someone tries to override them
       get block_class () { return 'nr-block' },
       get ignore_class () { return 'nr-ignore' },
@@ -55,14 +82,16 @@ const model = () => {
         return hiddenState.block_selector
       },
       set block_selector (val) {
-        hiddenState.block_selector += `,${val}`
+        if (isValidSelector(val)) hiddenState.block_selector += `,${val}`
+        else if (val !== '') warn('An invalid session_replay.block_selector was provided and will not be used', val)
       },
       // password: must always be present and true no matter what customer sets
       get mask_input_options () {
         return hiddenState.mask_input_options
       },
       set mask_input_options (val) {
-        hiddenState.mask_input_options = { ...val, password: true }
+        if (val && typeof val === 'object') hiddenState.mask_input_options = { ...val, password: true }
+        else warn('An invalid session_replay.mask_input_option was provided and will not be used', val)
       }
     },
     spa: { enabled: true, harvestTimeSeconds: 10, autoStart: true }

package/src/common/config/state/init.test.js

@@ -26,3 +26,43 @@ test('getConfigurationValue parses path correctly', () => {
   expect(getConfigurationValue('ab', 'page_action')).toEqual({ enabled: true, harvestTimeSeconds: 1000, autoStart: true })
   expect(getConfigurationValue('ab', 'page_action.harvestTimeSeconds')).toEqual(1000)
 })
+
+describe('property getters/setters used for validation', () => {
+  test('invalid values do not pass through', () => {
+    setConfiguration('12345', {
+      session_replay: {
+        block_selector: '[invalid selector]',
+        mask_text_selector: '[invalid selector]',
+        mask_input_options: 'select:true'
+      }
+    })
+
+    expect(getConfigurationValue('12345', 'session_replay.block_selector')).toEqual('[data-nr-block]')
+    expect(getConfigurationValue('12345', 'session_replay.mask_text_selector')).toEqual('*')
+    expect(getConfigurationValue('12345', 'session_replay.mask_input_options')).toMatchObject({ password: true, select: false })
+  })
+
+  test('valid values do pass through', () => {
+    setConfiguration('23456', {
+      session_replay: {
+        block_selector: '[block-text-test]',
+        mask_text_selector: '[mask-text-test]',
+        mask_input_options: { select: true }
+      }
+    })
+
+    expect(getConfigurationValue('23456', 'session_replay.block_selector')).toEqual('[data-nr-block],[block-text-test]')
+    expect(getConfigurationValue('23456', 'session_replay.mask_text_selector')).toEqual('[mask-text-test],[data-nr-mask]')
+    expect(getConfigurationValue('23456', 'session_replay.mask_input_options')).toMatchObject({ password: true, select: true })
+  })
+
+  test('null accepted for mask_text', () => {
+    setConfiguration('34567', {
+      session_replay: {
+        mask_text_selector: null
+      }
+    })
+
+    expect(getConfigurationValue('34567', 'session_replay.mask_text_selector')).toEqual(null)
+  })
+})

package/src/common/dom/query-selector.js

@@ -0,0 +1,9 @@
+export const isValidSelector = (selector) => {
+  if (!selector || typeof selector !== 'string') return false
+  try {
+    document.createDocumentFragment().querySelector(selector)
+  } catch {
+    return false
+  }
+  return true
+}

package/src/common/dom/query-selector.test.js

@@ -0,0 +1,24 @@
+import { isValidSelector } from './query-selector'
+describe('query selector tests', () => {
+  test('handles nullish values', () => {
+    expect(isValidSelector(null)).toEqual(false)
+    expect(isValidSelector('')).toEqual(false)
+    expect(isValidSelector(0)).toEqual(false)
+    expect(isValidSelector(false)).toEqual(false)
+  })
+
+  test('handles truthy but invalid values', () => {
+    expect(isValidSelector({ test: 1 })).toEqual(false)
+    expect(isValidSelector([1, 2, 3])).toEqual(false)
+    expect(isValidSelector(1)).toEqual(false)
+    expect(isValidSelector(',')).toEqual(false)
+    expect(isValidSelector('[invalid space]')).toEqual(false)
+  })
+
+  test('handles valid selectors', () => {
+    expect(isValidSelector('#id')).toEqual(true)
+    expect(isValidSelector('.class')).toEqual(true)
+    expect(isValidSelector('.multiple,#selectors')).toEqual(true)
+    expect(isValidSelector('[attr-selector]')).toEqual(true)
+  })
+})

package/src/features/session_replay/aggregate/index.js

@@ -21,6 +21,8 @@ import { sharedChannel } from '../../../common/constants/shared-channel'
 import { obj as encodeObj } from '../../../common/url/encode'
 import { warn } from '../../../common/util/console'
 import { globalScope } from '../../../common/constants/runtime'
+import { SUPPORTABILITY_METRIC_CHANNEL } from '../../metrics/constants'
+import { FEATURE_NAMES } from '../../../loaders/features/features'
 
 // would be better to get this dynamically in some way
 export const RRWEB_VERSION = '2.0.0-alpha.8'
@@ -91,7 +93,7 @@ export class Aggregate extends AggregateBase {
     if (shouldSetup) {
       // The SessionEntity class can emit a message indicating the session was cleared and reset (expiry, inactivity). This feature must abort and never resume if that occurs.
       this.ee.on(SESSION_EVENTS.RESET, () => {
-        this.abort()
+        this.abort('Session Reset')
       })
 
       // The SessionEntity class can emit a message indicating the session was paused (visibility change). This feature must stop recording if that occurs.
@@ -176,7 +178,7 @@ export class Aggregate extends AggregateBase {
       // Do not change the webpackChunkName or it will break the webpack nrba-chunking plugin
       recorder = (await import(/* webpackChunkName: "recorder" */'rrweb')).record
     } catch (err) {
-      return this.abort()
+      return this.abort('Recorder failed to import')
     }
 
     // FULL mode records AND reports from the beginning, while ERROR mode only records (but does not report).
@@ -251,7 +253,7 @@ export class Aggregate extends AggregateBase {
   onHarvestFinished (result) {
     // The mutual decision for now is to stop recording and clear buffers if ingest is experiencing 429 rate limiting
     if (result.status === 429) {
-      this.abort()
+      this.abort('429: Too many requests')
     }
 
     if (this.blocked) this.scheduler.stopTimer(true)
@@ -272,7 +274,7 @@ export class Aggregate extends AggregateBase {
   startRecording () {
     if (!recorder) {
       warn('Recording library was never imported')
-      return this.abort()
+      return this.abort('Recorder was never imported')
     }
     this.clearTimestamps()
     // set the fallbacks as early as possible
@@ -309,7 +311,8 @@ export class Aggregate extends AggregateBase {
     // Vortex will block payloads at a certain size, we might as well not send.
     if (payloadSize > MAX_PAYLOAD_SIZE) {
       this.clearBuffer()
-      return this.abort()
+      this.ee.emit(SUPPORTABILITY_METRIC_CHANNEL, ['SessionReplay/Too-Big/Seen'], undefined, FEATURE_NAMES.metrics, this.ee)
+      return this.abort('Payload too big')
     }
     // Checkout events are flags by the recording lib that indicate a fullsnapshot was taken every n ms. These are important
     // to help reconstruct the replay later and must be included.  While waiting and buffering for errors to come through,
@@ -373,7 +376,8 @@ export class Aggregate extends AggregateBase {
   }
 
   /** Abort the feature, once aborted it will not resume */
-  abort () {
+  abort (reason) {
+    warn(`SR aborted -- ${reason}`)
     this.blocked = true
     this.mode = MODE.OFF
     this.stopRecording()

package/src/features/utils/instrument-base.js

@@ -108,6 +108,7 @@ export class InstrumentBase extends FeatureBase {
         warn(`Downloading and initializing ${this.featureName} failed...`, e)
         this.abortHandler?.() // undo any important alterations made to the page
         // not supported yet but nice to do: "abort" this agent's EE for this feature specifically
+        drain(this.agentIdentifier, this.featureName)
         loadedSuccessfully(false)
       }
     }