@newrelic/browser-agent 1.248.0 → 1.250.0

package/CHANGELOG.md

@@ -3,6 +3,33 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [1.250.0](https://github.com/newrelic/newrelic-browser-agent/compare/v1.249.0...v1.250.0) (2024-01-09)
+
+
+### Features
+
+* Add a flag to note that a payload occurred during a replay ([#834](https://github.com/newrelic/newrelic-browser-agent/issues/834)) ([1b898c6](https://github.com/newrelic/newrelic-browser-agent/commit/1b898c61fa260bd0df644b4825ddb6d1c9b54b51))
+* Session Replay Dynamic Loading ([#832](https://github.com/newrelic/newrelic-browser-agent/issues/832)) ([1af7b89](https://github.com/newrelic/newrelic-browser-agent/commit/1af7b896ef41c8b4855ca85e2a4e40c20a8cdfb1))
+
+
+### Bug Fixes
+
+* TraceId generation regression (non-random trace ids) ([#853](https://github.com/newrelic/newrelic-browser-agent/issues/853)) ([8767f69](https://github.com/newrelic/newrelic-browser-agent/commit/8767f69122853a4099ed25f5886a91de06703c98))
+
+## [1.249.0](https://github.com/newrelic/newrelic-browser-agent/compare/v1.248.0...v1.249.0) (2023-12-14)
+
+
+### Features
+
+* Allow empty string for Session Replay masking value ([#831](https://github.com/newrelic/newrelic-browser-agent/issues/831)) ([34f837f](https://github.com/newrelic/newrelic-browser-agent/commit/34f837f65ab31a67b821f125e20e80d39d7790a9))
+
+
+### Bug Fixes
+
+* Fix API Warning Messages ([#830](https://github.com/newrelic/newrelic-browser-agent/issues/830)) ([2b13a0f](https://github.com/newrelic/newrelic-browser-agent/commit/2b13a0fdfad529dc1cfff43506e28473498ce8a1))
+* loader missing sub-resource integrity hashes ([#837](https://github.com/newrelic/newrelic-browser-agent/issues/837)) ([a9b6f2e](https://github.com/newrelic/newrelic-browser-agent/commit/a9b6f2e578b1684dd50f8eb491251c03eca88a12))
+* traceids not random when using `webcrypto` ([#825](https://github.com/newrelic/newrelic-browser-agent/issues/825)) ([e264acf](https://github.com/newrelic/newrelic-browser-agent/commit/e264acfbff2cacc93fae88daea70be3c1e006f90))
+
 ## [1.248.0](https://github.com/newrelic/newrelic-browser-agent/compare/v1.247.0...v1.248.0) (2023-11-16)
 
 

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

@@ -11,6 +11,7 @@ var _constants = require("../../session/constants");
 var _console = require("../../util/console");
 var _nreum = require("../../window/nreum");
 var _configurable = require("./configurable");
+const nrMask = '[data-nr-mask]';
 const model = () => {
   const hiddenState = {
     mask_selector: '*',
@@ -114,7 +115,7 @@ const model = () => {
       collect_fonts: false,
       // serialize fonts for collection without public asset url, this is currently broken in RRWeb -- https://github.com/rrweb-io/rrweb/issues/1304.  When fixed, revisit with test cases
       inline_images: false,
-      // serialize images for collection without public asset url
+      // serialize images for collection without public asset url -- right now this is only useful for testing as it easily generates payloads too large to be harvested
       inline_stylesheet: true,
       // serialize css for collection without public asset url
       // recording config settings
@@ -124,8 +125,7 @@ const model = () => {
         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);
+        if ((0, _querySelector.isValidSelector)(val)) hiddenState.mask_selector = "".concat(val, ",").concat(nrMask);else if (val === '' || val === null) hiddenState.mask_selector = nrMask;else (0, _console.warn)('An invalid session_replay.mask_selector was provided. \'*\' will 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() {

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 = "1.248.0";
+const VERSION = "1.250.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 = "1.248.0";
+const VERSION = "1.250.0";
 
 /**
  * Exposes the build type of the agent

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

@@ -8,7 +8,7 @@ var submitData = _interopRequireWildcard(require("../util/submit-data"));
 var _sharedContext = require("../context/shared-context");
 var _harvest = require("./harvest");
 var _eol = require("../unload/eol");
-var _sessionEntity = require("../session/session-entity");
+var _constants = require("../session/constants");
 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; }
 /*
@@ -47,7 +47,7 @@ class HarvestScheduler extends _sharedContext.SharedContext {
     /* Flush all buffered data if session resets and give up retries. This should be synchronous to ensure that the correct `session` value is sent.
       Since session-reset generates a new session ID and the ID is grabbed at send-time, any delays or retries would cause the payload to be sent under
       the wrong session ID. */
-    this.sharedContext?.ee.on(_sessionEntity.SESSION_EVENTS.RESET, () => this.runHarvest({
+    this.sharedContext?.ee.on(_constants.SESSION_EVENTS.RESET, () => this.runHarvest({
       forceNoRetry: true
     }));
   }

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

@@ -137,7 +137,7 @@ class Harvest extends _sharedContext.SharedContext {
     let url = "".concat(protocol, "://").concat(perceviedBeacon).concat(endpointURLPart, "/1/").concat(info.licenseKey);
     if (customUrl) url = customUrl;
     if (raw) url = "".concat(protocol, "://").concat(perceviedBeacon, "/").concat(endpoint);
-    const baseParams = !raw && includeBaseParams ? this.baseQueryString() : '';
+    const baseParams = !raw && includeBaseParams ? this.baseQueryString(qs) : '';
     let payloadParams = (0, _encode.obj)(qs, agentRuntime.maxBytes);
     if (!submitMethod) {
       submitMethod = submitData.getSubmitMethod({
@@ -202,7 +202,7 @@ class Harvest extends _sharedContext.SharedContext {
   }
 
   // The stuff that gets sent every time.
-  baseQueryString() {
+  baseQueryString(qs) {
     const runtime = (0, _config.getRuntime)(this.sharedContext.agentIdentifier);
     const info = (0, _config.getInfo)(this.sharedContext.agentIdentifier);
     const location = (0, _cleanUrl.cleanURL)((0, _location.getLocation)());
@@ -211,7 +211,8 @@ class Harvest extends _sharedContext.SharedContext {
     // ck param DEPRECATED - still expected by backend
     '&s=' + (runtime.session?.state.value || '0'),
     // the 0 id encaps all untrackable and default traffic
-    (0, _encode.param)('ref', ref), (0, _encode.param)('ptid', runtime.ptid ? '' + runtime.ptid : '')].join('');
+    (0, _encode.param)('ref', ref), (0, _encode.param)('ptid', runtime.ptid ? '' + runtime.ptid : ''), (0, _encode.param)('hr', runtime?.session?.state.sessionReplayMode === 1 ? '1' : '0', qs) // hasReplay
+    ].join('');
   }
 
   /**

package/dist/cjs/common/ids/unique-id.js

@@ -50,12 +50,13 @@ function generateUuid() {
   let randomValueTable;
   let randomValueIndex = 0;
   if (crypto && crypto.getRandomValues) {
+    // For a UUID, we only need 30 characters since two characters are pre-defined
     // eslint-disable-next-line
-    randomValueTable = crypto.getRandomValues(new Uint8Array(31));
+    randomValueTable = crypto.getRandomValues(new Uint8Array(30));
   }
   return uuidv4Template.split('').map(templateInput => {
     if (templateInput === 'x') {
-      return getRandomValue(randomValueTable, ++randomValueIndex).toString(16);
+      return getRandomValue(randomValueTable, randomValueIndex++).toString(16);
     } else if (templateInput === 'y') {
       // this is the uuid variant per spec (8, 9, a, b)
       // % 4, then shift to get values 8-11
@@ -78,11 +79,11 @@ function generateRandomHexString(length) {
   let randomValueIndex = 0;
   if (crypto && crypto.getRandomValues) {
     // eslint-disable-next-line
-    randomValueTable = crypto.getRandomValues(new Uint8Array(31));
+    randomValueTable = crypto.getRandomValues(new Uint8Array(length));
   }
   const chars = [];
   for (var i = 0; i < length; i++) {
-    chars.push(getRandomValue(randomValueTable, ++randomValueIndex).toString(16));
+    chars.push(getRandomValue(randomValueTable, randomValueIndex++).toString(16));
   }
   return chars.join('');
 }

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

@@ -3,10 +3,28 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.PREFIX = exports.DEFAULT_INACTIVE_MS = exports.DEFAULT_EXPIRES_MS = void 0;
+exports.SESSION_EVENT_TYPES = exports.SESSION_EVENTS = exports.PREFIX = exports.MODE = exports.DEFAULT_INACTIVE_MS = exports.DEFAULT_EXPIRES_MS = void 0;
 const PREFIX = 'NRBA';
 exports.PREFIX = PREFIX;
 const DEFAULT_EXPIRES_MS = 14400000;
 exports.DEFAULT_EXPIRES_MS = DEFAULT_EXPIRES_MS;
 const DEFAULT_INACTIVE_MS = 1800000;
-exports.DEFAULT_INACTIVE_MS = DEFAULT_INACTIVE_MS;
+exports.DEFAULT_INACTIVE_MS = DEFAULT_INACTIVE_MS;
+const SESSION_EVENTS = {
+  PAUSE: 'session-pause',
+  RESET: 'session-reset',
+  RESUME: 'session-resume',
+  UPDATE: 'session-update'
+};
+exports.SESSION_EVENTS = SESSION_EVENTS;
+const SESSION_EVENT_TYPES = {
+  SAME_TAB: 'same-tab',
+  CROSS_TAB: 'cross-tab'
+};
+exports.SESSION_EVENT_TYPES = SESSION_EVENT_TYPES;
+const MODE = {
+  OFF: 0,
+  FULL: 1,
+  ERROR: 2
+};
+exports.MODE = MODE;

package/dist/cjs/common/session/session-entity.js

@@ -3,7 +3,7 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.SessionEntity = exports.SESSION_EVENT_TYPES = exports.SESSION_EVENTS = exports.MODE = void 0;
+exports.SessionEntity = void 0;
 var _uniqueId = require("../ids/unique-id");
 var _console = require("../util/console");
 var _stringify = require("../util/stringify");
@@ -18,37 +18,19 @@ var _handle = require("../event-emitter/handle");
 var _constants2 = require("../../features/metrics/constants");
 var _features = require("../../loaders/features/features");
 var _eventListenerOpts = require("../event-listener/event-listener-opts");
-const MODE = {
-  OFF: 0,
-  FULL: 1,
-  ERROR: 2
-};
 // this is what can be stored in local storage (not enforced but probably should be)
 // these values should sync between local storage and the parent class props
-exports.MODE = MODE;
 const model = {
   value: '',
   inactiveAt: 0,
   expiresAt: 0,
   updatedAt: Date.now(),
-  sessionReplayMode: MODE.OFF,
+  sessionReplayMode: _constants.MODE.OFF,
   sessionReplaySentFirstChunk: false,
-  sessionTraceMode: MODE.OFF,
+  sessionTraceMode: _constants.MODE.OFF,
   traceHarvestStarted: false,
   custom: {}
 };
-const SESSION_EVENTS = {
-  PAUSE: 'session-pause',
-  RESET: 'session-reset',
-  RESUME: 'session-resume',
-  UPDATE: 'session-update'
-};
-exports.SESSION_EVENTS = SESSION_EVENTS;
-const SESSION_EVENT_TYPES = {
-  SAME_TAB: 'same-tab',
-  CROSS_TAB: 'cross-tab'
-};
-exports.SESSION_EVENT_TYPES = SESSION_EVENT_TYPES;
 class SessionEntity {
   /**
    * Create a self-managing Session Entity. This entity is scoped to the agent identifier which triggered it, allowing for multiple simultaneous session objects to exist.
@@ -79,7 +61,7 @@ class SessionEntity {
         if (event.key === this.lookupKey) {
           const obj = typeof event.newValue === 'string' ? JSON.parse(event.newValue) : event.newValue;
           this.sync(obj);
-          this.ee.emit(SESSION_EVENTS.UPDATE, [SESSION_EVENT_TYPES.CROSS_TAB, this.state]);
+          this.ee.emit(_constants.SESSION_EVENTS.UPDATE, [_constants.SESSION_EVENT_TYPES.CROSS_TAB, this.state]);
         }
       });
     }
@@ -137,11 +119,11 @@ class SessionEntity {
         // When the inactive timer refreshes, it will update the storage values with an update timestamp
         onRefresh: this.refresh.bind(this),
         onResume: () => {
-          this.ee.emit(SESSION_EVENTS.RESUME);
+          this.ee.emit(_constants.SESSION_EVENTS.RESUME);
         },
         // When the inactive timer pauses, update the storage values with an update timestamp
         onPause: () => {
-          if (this.initialized) this.ee.emit(SESSION_EVENTS.PAUSE);
+          if (this.initialized) this.ee.emit(_constants.SESSION_EVENTS.PAUSE);
           this.write((0, _configurable.getModeledObject)(this.state, model));
         },
         ee: this.ee,
@@ -216,7 +198,7 @@ class SessionEntity {
       //
       // TODO - compression would need happen here if we decide to do it
       this.storage.set(this.lookupKey, (0, _stringify.stringify)(this.state));
-      this.ee.emit(SESSION_EVENTS.UPDATE, [SESSION_EVENT_TYPES.SAME_TAB, this.state]);
+      this.ee.emit(_constants.SESSION_EVENTS.UPDATE, [_constants.SESSION_EVENT_TYPES.SAME_TAB, this.state]);
       return data;
     } catch (e) {
       // storage is inaccessible
@@ -230,7 +212,7 @@ class SessionEntity {
     // * stop recording (stn and sr)...
     // * delete the session and start over
     try {
-      if (this.initialized) this.ee.emit(SESSION_EVENTS.RESET);
+      if (this.initialized) this.ee.emit(_constants.SESSION_EVENTS.RESET);
       this.storage.remove(this.lookupKey);
       this.inactiveTimer?.abort?.();
       this.expiresTimer?.clear?.();

package/dist/cjs/common/url/encode.js

@@ -72,6 +72,8 @@ function obj(payload, maxBytes) {
 
 // Constructs an HTTP parameter to add to the BAM router URL
 function param(name, value) {
+  let base = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : {};
+  if (Object.keys(base).includes(name)) return ''; // we assume if feature supplied a matching qp to the base, we should honor what the feature sent over the default
   if (value && typeof value === 'string') {
     return '&' + name + '=' + qs(value);
   }

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

@@ -101,10 +101,12 @@ class Aggregate extends _aggregateBase.AggregateBase {
 
     // Check if proxy for either chunks or beacon is being used
     const {
-      proxy
+      proxy,
+      privacy
     } = (0, _config.getConfiguration)(this.agentIdentifier);
     if (proxy.assets) this.storeSupportabilityMetrics('Config/AssetsUrl/Changed');
     if (proxy.beacon) this.storeSupportabilityMetrics('Config/BeaconUrl/Changed');
+    if (!(_runtime.isBrowserScope && privacy.cookies_enabled)) this.storeSupportabilityMetrics('Config/SessionTracking/Disabled');
   }
   eachSessionChecks() {
     if (!_runtime.isBrowserScope) return;