@newrelic/browser-agent 1.232.0 → 1.232.1

package/dist/cjs/features/jserrors/aggregate/compute-stack-trace.test.js

@@ -2,16 +2,31 @@
 
 var _faker = require("@faker-js/faker");
 var _testingUtils = require("../../../../tools/testing-utils");
-var _computeStackTrace = require("./compute-stack-trace");
+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; }
+const globalScopeLocation = 'https://example.com/';
+const mockGlobalScopeLocation = url => {
+  jest.doMock('../../../common/util/global-scope', () => ({
+    initialLocation: url || globalScopeLocation
+  }));
+};
+afterEach(() => {
+  jest.resetModules();
+  jest.clearAllMocks();
+});
 const baseMockError = {
   toString: 'RangeError: Invalid array length',
   name: 'RangeError',
   constructor: 'function RangeError() { [native code] }',
   message: 'Invalid array length',
-  stack: 'RangeError: Invalid array length\n    at errorTest (http://bam-test-1.nr-local.net:3334/tests/assets/instrumented.html?loader=spa:74:16)\n    at captureError (http://bam-test-1.nr-local.net:3334/tests/assets/instrumented.html?loader=spa:17:9)\n    at onload (http://bam-test-1.nr-local.net:3334/tests/assets/instrumented.html?loader=spa:70:5)'
+  stack: 'RangeError: Invalid array length\n' + '    at errorTest (' + globalScopeLocation + '?loader=spa#hello:74:16)\n' + '    at captureError (' + globalScopeLocation + 'js/script.js?loader=spa:17:9)\n' + '    at onload (' + globalScopeLocation + 'js/script.js?loader=spa:70:5)'
 };
-test('parsing should return a failure for a null error object', () => {
-  const result = (0, _computeStackTrace.computeStackTrace)(null);
+test('parsing should return a failure for a null error object', async () => {
+  mockGlobalScopeLocation();
+  const {
+    computeStackTrace
+  } = await Promise.resolve().then(() => _interopRequireWildcard(require('./compute-stack-trace')));
+  const result = computeStackTrace(null);
   expect(result).toEqual(expect.objectContaining({
     mode: 'failed',
     stackString: '',
@@ -19,25 +34,46 @@ test('parsing should return a failure for a null error object', () => {
   }));
 });
 describe('errors with stack property', () => {
-  test('parsed name should be unknown when name and constructor are missing', () => {
+  test('should show <inline> for same-page stack string URLs but not sub-paths', async () => {
+    const mockError = _testingUtils.browserErrorUtils.constructError({
+      ...baseMockError
+    });
+    mockGlobalScopeLocation();
+    const {
+      computeStackTrace
+    } = await Promise.resolve().then(() => _interopRequireWildcard(require('./compute-stack-trace')));
+    const result = computeStackTrace(mockError);
+    expect(result).toEqual(expect.objectContaining({
+      stackString:
+      // canonicalized
+      'RangeError: Invalid array length\n' + '    at errorTest (<inline>:74:16)\n' + '    at captureError (' + globalScopeLocation + 'js/script.js:17:9)\n' + '    at onload (' + globalScopeLocation + 'js/script.js:70:5)'
+    }));
+  });
+  test('parsed name should be unknown when name and constructor are missing', async () => {
     const mockError = _testingUtils.browserErrorUtils.constructError({
       ...baseMockError,
       name: null,
       constructor: null
     });
-    const result = (0, _computeStackTrace.computeStackTrace)(mockError);
+    mockGlobalScopeLocation();
+    const {
+      computeStackTrace
+    } = await Promise.resolve().then(() => _interopRequireWildcard(require('./compute-stack-trace')));
+    const result = computeStackTrace(mockError);
     expect(result).toEqual(expect.objectContaining({
       mode: 'stack',
-      name: 'unknown',
-      message: mockError.message,
-      stackString: mockError.stack
+      name: 'unknown'
     }));
   });
-  test('parsed stack should not contain nrWrapper', () => {
+  test('parsed stack should not contain nrWrapper', async () => {
     const alteredError = baseMockError;
-    alteredError.stack += '\n    at nrWrapper (http://bam-test-1.nr-local.net:3334/tests/assets/instrumented.html?loader=spa:60:17)';
+    alteredError.stack += '\n    at nrWrapper (' + globalScopeLocation + '?loader=spa:60:17)';
     const mockError = _testingUtils.browserErrorUtils.constructError(alteredError);
-    const result = (0, _computeStackTrace.computeStackTrace)(mockError);
+    mockGlobalScopeLocation();
+    const {
+      computeStackTrace
+    } = await Promise.resolve().then(() => _interopRequireWildcard(require('./compute-stack-trace')));
+    const result = computeStackTrace(mockError);
     expect(result).toEqual(expect.objectContaining({
       mode: 'stack',
       name: mockError.name,
@@ -48,17 +84,21 @@ describe('errors with stack property', () => {
       func: 'nrWrapper'
     }));
   });
-  test('stack should still parse when column numbers are missing', () => {
+  test('stack should still parse when column numbers are missing', async () => {
     const mockError = _testingUtils.browserErrorUtils.constructError({
       ...baseMockError,
-      stack: 'Error: Blocked a frame with origin "http://bam-test-1.nr-local.net:3334" from accessing a cross-origin frame.\n    at errorTest (http://bam-test-1.nr-local.net:3334/tests/assets/instrumented.html?loader=spa:60)\n    at captureError (http://bam-test-1.nr-local.net:3334/tests/assets/instrumented.html?loader=spa:17)\n    at onload (http://bam-test-1.nr-local.net:3334/tests/assets/instrumented.html?loader=spa:57)'
+      stack: 'RangeError: Invalid array length\n' + 'Error: Blocked a frame with origin "http://bam-test-1.nr-local.net:3334" from accessing a cross-origin frame.\n' + '    at errorTest (' + globalScopeLocation + '?loader=spa:60)\n' + '    at captureError (' + globalScopeLocation + '?loader=spa:17)\n' + '    at onload (' + globalScopeLocation + '?loader=spa:57)'
     });
-    const result = (0, _computeStackTrace.computeStackTrace)(mockError);
+    mockGlobalScopeLocation();
+    const {
+      computeStackTrace
+    } = await Promise.resolve().then(() => _interopRequireWildcard(require('./compute-stack-trace')));
+    const result = computeStackTrace(mockError);
     expect(result).toEqual(expect.objectContaining({
       mode: 'stack',
       name: mockError.name,
       message: mockError.message,
-      stackString: mockError.stack
+      stackString: 'RangeError: Invalid array length\n' + 'Error: Blocked a frame with origin "http://bam-test-1.nr-local.net:3334" from accessing a cross-origin frame.\n' + '    at errorTest (<inline>:60)\n' + '    at captureError (<inline>:17)\n' + '    at onload (<inline>:57)'
     }));
     expect(result.frames.length).toEqual(3);
     expect(result.frames).toContainEqual(expect.objectContaining({
@@ -74,12 +114,16 @@ describe('errors with stack property', () => {
       column: null
     }));
   });
-  test('parser can handle chrome eval stack', () => {
+  test('parser can handle chrome eval stack', async () => {
     const mockError = _testingUtils.browserErrorUtils.constructError({
       ...baseMockError,
-      stack: '    at foobar (eval at foobar (http://bam-test-1.nr-local.net:3334/tests/assets/instrumented.html))'
+      stack: '    at foobar (eval at foobar (' + globalScopeLocation + '))'
     });
-    const result = (0, _computeStackTrace.computeStackTrace)(mockError);
+    mockGlobalScopeLocation();
+    const {
+      computeStackTrace
+    } = await Promise.resolve().then(() => _interopRequireWildcard(require('./compute-stack-trace')));
+    const result = computeStackTrace(mockError);
     expect(result).toEqual(expect.objectContaining({
       mode: 'stack',
       name: mockError.name,
@@ -91,7 +135,7 @@ describe('errors with stack property', () => {
       func: 'evaluated code'
     }));
   });
-  test('parser can handle ie eval stack', () => {
+  test('parser can handle ie eval stack', async () => {
     const mockError = _testingUtils.browserErrorUtils.constructError({
       toString: 'TypeError: Permission denied',
       name: 'TypeError',
@@ -99,7 +143,11 @@ describe('errors with stack property', () => {
       message: 'Permission denied',
       stack: '    at Function code (Function code:23:23)'
     });
-    const result = (0, _computeStackTrace.computeStackTrace)(mockError);
+    mockGlobalScopeLocation();
+    const {
+      computeStackTrace
+    } = await Promise.resolve().then(() => _interopRequireWildcard(require('./compute-stack-trace')));
+    const result = computeStackTrace(mockError);
     expect(result).toEqual(expect.objectContaining({
       mode: 'stack',
       name: mockError.name,
@@ -111,12 +159,16 @@ describe('errors with stack property', () => {
       func: 'evaluated code'
     }));
   });
-  test('parser can handle stack with anonymous function', () => {
+  test('parser can handle stack with anonymous function', async () => {
     const mockError = _testingUtils.browserErrorUtils.constructError({
       ...baseMockError,
       stack: 'anonymous'
     });
-    const result = (0, _computeStackTrace.computeStackTrace)(mockError);
+    mockGlobalScopeLocation();
+    const {
+      computeStackTrace
+    } = await Promise.resolve().then(() => _interopRequireWildcard(require('./compute-stack-trace')));
+    const result = computeStackTrace(mockError);
     expect(result).toEqual(expect.objectContaining({
       mode: 'stack',
       name: mockError.name,
@@ -133,7 +185,7 @@ describe('errors without stack property and with line property', () => {
   /**
    * @deprecated sourceURL is no longer present in errors for any browsers we support
    */
-  test('parsed stack should contain sourceURL and line number', () => {
+  test('parsed stack should contain sourceURL and line number', async () => {
     const sourceURL = _faker.faker.internet.url();
     const mockError = _testingUtils.browserErrorUtils.constructError({
       ...baseMockError,
@@ -141,7 +193,11 @@ describe('errors without stack property and with line property', () => {
       line: 100,
       sourceURL
     });
-    const result = (0, _computeStackTrace.computeStackTrace)(mockError);
+    mockGlobalScopeLocation();
+    const {
+      computeStackTrace
+    } = await Promise.resolve().then(() => _interopRequireWildcard(require('./compute-stack-trace')));
+    const result = computeStackTrace(mockError);
     expect(result).toEqual(expect.objectContaining({
       mode: 'sourceline',
       name: mockError.name,
@@ -158,7 +214,7 @@ describe('errors without stack property and with line property', () => {
   /**
    * @deprecated sourceURL is no longer present in errors for any browsers we support
    */
-  test('parsed stack should contain sourceURL, line number, and column number', () => {
+  test('parsed stack should contain sourceURL, line number, and column number', async () => {
     const sourceURL = _faker.faker.internet.url();
     const mockError = _testingUtils.browserErrorUtils.constructError({
       ...baseMockError,
@@ -167,7 +223,11 @@ describe('errors without stack property and with line property', () => {
       stack: undefined,
       sourceURL
     });
-    const result = (0, _computeStackTrace.computeStackTrace)(mockError);
+    mockGlobalScopeLocation();
+    const {
+      computeStackTrace
+    } = await Promise.resolve().then(() => _interopRequireWildcard(require('./compute-stack-trace')));
+    const result = computeStackTrace(mockError);
     expect(result).toEqual(expect.objectContaining({
       mode: 'sourceline',
       name: mockError.name,
@@ -181,14 +241,18 @@ describe('errors without stack property and with line property', () => {
       column: mockError.column
     }));
   });
-  test('parsed stack should contain "evaluated code" if sourceURL property is not present', () => {
+  test('parsed stack should contain "evaluated code" if sourceURL property is not present', async () => {
     const mockError = _testingUtils.browserErrorUtils.constructError({
       ...baseMockError,
       line: 100,
       column: 200,
       stack: undefined
     });
-    const result = (0, _computeStackTrace.computeStackTrace)(mockError);
+    mockGlobalScopeLocation();
+    const {
+      computeStackTrace
+    } = await Promise.resolve().then(() => _interopRequireWildcard(require('./compute-stack-trace')));
+    const result = computeStackTrace(mockError);
     expect(result).toEqual(expect.objectContaining({
       mode: 'sourceline',
       name: mockError.name,
@@ -201,6 +265,58 @@ describe('errors without stack property and with line property', () => {
     }));
   });
 
+  /**
+   * @deprecated sourceURL is no longer present in errors for any browsers we support
+   */
+  test('should show <inline> for same-page URLs', async () => {
+    const pageLocation = _faker.faker.internet.url();
+    const sourceURL = pageLocation + '?abc=123';
+    const mockError = _testingUtils.browserErrorUtils.constructError({
+      ...baseMockError,
+      line: 100,
+      column: 200,
+      stack: undefined,
+      sourceURL: sourceURL
+    });
+    mockGlobalScopeLocation(pageLocation);
+    const {
+      computeStackTrace
+    } = await Promise.resolve().then(() => _interopRequireWildcard(require('./compute-stack-trace')));
+    const result = computeStackTrace(mockError);
+    expect(result).toEqual(expect.objectContaining({
+      stackString: "".concat(mockError.name, ": ").concat(mockError.message, "\n    at <inline>:").concat(mockError.line, ":").concat(mockError.column)
+    }));
+    expect(result.frames).toContainEqual(expect.objectContaining({
+      url: '<inline>'
+    }));
+  });
+
+  /**
+   * @deprecated sourceURL is no longer present in errors for any browsers we support
+   */
+  test('should NOT show <inline> for same-domain URLs with a sub-path', async () => {
+    const pageLocation = _faker.faker.internet.url();
+    const sourceURL = pageLocation + '/path/to/script.js';
+    const mockError = _testingUtils.browserErrorUtils.constructError({
+      ...baseMockError,
+      line: 100,
+      column: 200,
+      stack: undefined,
+      sourceURL
+    });
+    mockGlobalScopeLocation(pageLocation);
+    const {
+      computeStackTrace
+    } = await Promise.resolve().then(() => _interopRequireWildcard(require('./compute-stack-trace')));
+    const result = computeStackTrace(mockError);
+    expect(result).toEqual(expect.objectContaining({
+      stackString: "".concat(mockError.name, ": ").concat(mockError.message, "\n    at ").concat(sourceURL, ":").concat(mockError.line, ":").concat(mockError.column)
+    }));
+    expect(result.frames).toContainEqual(expect.objectContaining({
+      url: sourceURL
+    }));
+  });
+
   // TODO: computeStackTraceBySourceAndLine does not respect firefox lineNumber and columnNumber properties when stack is empty
 });
 
@@ -209,12 +325,16 @@ describe('errors without stack property and with line property', () => {
  * error, including primitives.
  */
 describe('errors that are messages only or primitives', () => {
-  test('parser should get error name from constructor', () => {
+  test('parser should get error name from constructor', async () => {
     const mockError = _testingUtils.browserErrorUtils.constructError({
       toString: '0',
       constructor: 'function Number() { [native code] }'
     });
-    const result = (0, _computeStackTrace.computeStackTrace)(mockError);
+    mockGlobalScopeLocation();
+    const {
+      computeStackTrace
+    } = await Promise.resolve().then(() => _interopRequireWildcard(require('./compute-stack-trace')));
+    const result = computeStackTrace(mockError);
     expect(result).toEqual(expect.objectContaining({
       mode: 'nameonly',
       name: 'Number',
@@ -222,13 +342,17 @@ describe('errors that are messages only or primitives', () => {
       frames: []
     }));
   });
-  test('parser should get error name from name property', () => {
+  test('parser should get error name from name property', async () => {
     const mockError = _testingUtils.browserErrorUtils.constructError({
       toString: '0',
       name: _faker.faker.datatype.uuid(),
       constructor: 'function Number() { [native code] }'
     });
-    const result = (0, _computeStackTrace.computeStackTrace)(mockError);
+    mockGlobalScopeLocation();
+    const {
+      computeStackTrace
+    } = await Promise.resolve().then(() => _interopRequireWildcard(require('./compute-stack-trace')));
+    const result = computeStackTrace(mockError);
     expect(result).toEqual(expect.objectContaining({
       mode: 'nameonly',
       name: mockError.name,
@@ -236,14 +360,18 @@ describe('errors that are messages only or primitives', () => {
       frames: []
     }));
   });
-  test('parser should include the message property', () => {
+  test('parser should include the message property', async () => {
     const mockError = _testingUtils.browserErrorUtils.constructError({
       toString: '0',
       name: _faker.faker.datatype.uuid(),
       message: _faker.faker.datatype.uuid(),
       constructor: 'function Number() { [native code] }'
     });
-    const result = (0, _computeStackTrace.computeStackTrace)(mockError);
+    mockGlobalScopeLocation();
+    const {
+      computeStackTrace
+    } = await Promise.resolve().then(() => _interopRequireWildcard(require('./compute-stack-trace')));
+    const result = computeStackTrace(mockError);
     expect(result).toEqual(expect.objectContaining({
       mode: 'nameonly',
       name: mockError.name,
@@ -252,6 +380,4 @@ describe('errors that are messages only or primitives', () => {
       frames: []
     }));
   });
-});
-
-// describe('')
+});

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

@@ -5,7 +5,6 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.Aggregate = void 0;
 var _canonicalFunctionName = require("./canonical-function-name");
-var _cleanUrl = require("../../../common/url/clean-url");
 var _computeStackTrace = require("./compute-stack-trace");
 var _stringHashCode = require("./string-hash-code");
 var _formatStackTrace = require("./format-stack-trace");
@@ -26,6 +25,10 @@ var _featureBase = require("../../utils/feature-base");
  * SPDX-License-Identifier: Apache-2.0
  */
 
+/**
+ * @typedef {import('./compute-stack-trace.js').StackInfo} StackInfo
+ */
+
 class Aggregate extends _featureBase.FeatureBase {
   static featureName = _constants.FEATURE_NAME;
   constructor(agentIdentifier, aggregator) {
@@ -109,49 +112,25 @@ class Aggregate extends _featureBase.FeatureBase {
   getBucketName(params, customParams) {
     return this.nameHash(params) + ':' + (0, _stringHashCode.stringHashCode)((0, _stringify.stringify)(customParams));
   }
-  canonicalizeURL(url, cleanedOrigin) {
-    if (typeof url !== 'string') return '';
-    var cleanedURL = (0, _cleanUrl.cleanURL)(url);
-    if (cleanedURL === cleanedOrigin) {
-      return '<inline>';
-    } else {
-      return cleanedURL;
-    }
-  }
-  buildCanonicalStackString(stackInfo, cleanedOrigin) {
-    var canonicalStack = '';
-    for (var i = 0; i < stackInfo.frames.length; i++) {
-      var frame = stackInfo.frames[i];
-      var func = (0, _canonicalFunctionName.canonicalFunctionName)(frame.func);
-      if (canonicalStack) canonicalStack += '\n';
-      if (func) canonicalStack += func + '@';
-      if (typeof frame.url === 'string') canonicalStack += frame.url;
-      if (frame.line) canonicalStack += ':' + frame.line;
-    }
-    return canonicalStack;
-  }
 
-  // Strip query parameters and fragments from the stackString property of the
-  // given stackInfo, along with the 'url' properties of each frame in
-  // stackInfo.frames.
-  //
-  // Any URLs that are equivalent to the cleaned version of the origin will also
-  // be replaced with the string '<inline>'.
-  //
-  canonicalizeStackURLs(stackInfo) {
-    // Currently, loader.origin might contain a fragment, but we don't want to use it
-    // for comparing with frame URLs.
-    var cleanedOrigin = (0, _cleanUrl.cleanURL)((0, _config.getRuntime)(this.agentIdentifier).origin);
+  /**
+   * Builds a standardized stack trace string from the frames in the given `stackInfo` object, with each frame separated
+   * by a newline character. Lines take the form `<functionName>@<url>:<lineNumber>`.
+   *
+   * @param {StackInfo} stackInfo - An object specifying a stack string and individual frames.
+   * @returns {string} A canonical stack string built from the URLs and function names in the given `stackInfo` object.
+   */
+  buildCanonicalStackString(stackInfo) {
+    var canonicalStackString = '';
     for (var i = 0; i < stackInfo.frames.length; i++) {
       var frame = stackInfo.frames[i];
-      var originalURL = frame.url;
-      var cleanedURL = this.canonicalizeURL(originalURL, cleanedOrigin);
-      if (cleanedURL && cleanedURL !== frame.url) {
-        frame.url = cleanedURL;
-        stackInfo.stackString = stackInfo.stackString.split(originalURL).join(cleanedURL);
-      }
+      var func = (0, _canonicalFunctionName.canonicalFunctionName)(frame.func);
+      if (canonicalStackString) canonicalStackString += '\n';
+      if (func) canonicalStackString += func + '@';
+      if (typeof frame.url === 'string') canonicalStackString += frame.url;
+      if (frame.line) canonicalStackString += ':' + frame.line;
     }
-    return stackInfo;
+    return canonicalStackString;
   }
   storeError(err, time, internal, customAttributes) {
     // are we in an interaction
@@ -168,10 +147,10 @@ class Aggregate extends _featureBase.FeatureBase {
       // Again as with previous usage, all falsey values would include the error.
     }
 
-    var stackInfo = this.canonicalizeStackURLs((0, _computeStackTrace.computeStackTrace)(err));
-    var canonicalStack = this.buildCanonicalStackString(stackInfo);
+    var stackInfo = (0, _computeStackTrace.computeStackTrace)(err);
+    var canonicalStackString = this.buildCanonicalStackString(stackInfo);
     const params = {
-      stackHash: (0, _stringHashCode.stringHashCode)(canonicalStack),
+      stackHash: (0, _stringHashCode.stringHashCode)(canonicalStackString),
       exceptionClass: stackInfo.name,
       request_uri: _globalScope.globalScope?.location.pathname
     };

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

@@ -8,7 +8,6 @@ var _handle = require("../../../common/event-emitter/handle");
 var _now = require("../../../common/timing/now");
 var _getOrSet = require("../../../common/util/get-or-set");
 var _wrap = require("../../../common/wrap");
-var _lodash = _interopRequireDefault(require("lodash._slice"));
 require("./debug");
 var _instrumentBase = require("../../utils/instrument-base");
 var _constants = require("../constants");
@@ -17,7 +16,6 @@ var _globalScope = require("../../../common/util/global-scope");
 var _eventListenerOpts = require("../../../common/event-listener/event-listener-opts");
 var _config = require("../../../common/config/config");
 var _stringify = require("../../../common/util/stringify");
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 /*
  * Copyright 2020 New Relic Corporation. All rights reserved.
  * SPDX-License-Identifier: Apache-2.0

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

@@ -10,14 +10,12 @@ var _mapOwn = require("../../../common/util/map-own");
 var _stringify = require("../../../common/util/stringify");
 var _parseUrl = require("../../../common/url/parse-url");
 var _supportsPerformanceObserver = require("../../../common/window/supports-performance-observer");
-var _lodash = _interopRequireDefault(require("lodash._slice"));
 var _config = require("../../../common/config/config");
 var _now = require("../../../common/timing/now");
 var _constants = require("../constants");
 var _drain = require("../../../common/drain/drain");
 var _handlerCache = require("../../utils/handler-cache");
 var _featureBase = require("../../utils/feature-base");
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 /*
  * Copyright 2020 New Relic Corporation. All rights reserved.
  * SPDX-License-Identifier: Apache-2.0
@@ -260,7 +258,7 @@ class Aggregate extends _featureBase.FeatureBase {
     } else if (t && typeof t.tagName === 'string') {
       origin = t.tagName.toLowerCase();
       if (t.id) origin += '#' + t.id;
-      if (t.className) origin += '.' + (0, _lodash.default)(t.classList).join('.');
+      if (t.className) origin += '.' + Array.from(t.classList).join('.');
     }
     if (origin === 'unknown') {
       if (typeof target === 'string') origin = target;else if (target === document) origin = 'document';else if (target === window) origin = 'window';else if (target instanceof FileReader) origin = 'FileReader';

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

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

package/dist/esm/common/url/canonicalize-url.js

@@ -0,0 +1,27 @@
+/*
+ * Copyright 2020 New Relic Corporation. All rights reserved.
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+import { initialLocation } from '../util/global-scope';
+import { cleanURL } from './clean-url';
+
+/**
+ * Converts a URL to its basic form without a query string or fragment. If the resulting URL is the same as the
+ * loader's origin URL, returns '<inline>'.
+ * @param {string} url - The URL to be canonicalized.
+ * @param {string} loaderOriginUrl - The origin URL of the agent loader, used for inline detection.
+ * @returns {string} The canonicalized URL, or '<inline>' if the URL matches the loader origin URL.
+ */
+export function canonicalizeUrl(url) {
+  if (typeof url !== 'string') return '';
+  const cleanedUrl = cleanURL(url);
+  const cleanedGlobalScopeUrl = cleanURL(initialLocation);
+
+  // If the URL matches the origin URL of the loader, we assume it originated within an inline script.
+  if (cleanedUrl === cleanedGlobalScopeUrl) {
+    return '<inline>';
+  } else {
+    return cleanedUrl;
+  }
+}

package/dist/esm/common/url/canonicalize-url.test.js

@@ -0,0 +1,38 @@
+afterEach(() => {
+  jest.resetModules();
+});
+test.each([null, undefined, 34])('returns empty string when url argument is %s', async url => {
+  const {
+    canonicalizeUrl
+  } = await import('./canonicalize-url');
+  expect(canonicalizeUrl(url)).toEqual('');
+});
+test('strips URLs of query strings and fragments', async () => {
+  jest.doMock('../util/global-scope', () => ({
+    initialLocation: 'http://different-domain.com/'
+  }));
+  const {
+    canonicalizeUrl
+  } = await import('./canonicalize-url');
+  expect(canonicalizeUrl('http://example.com/path?query=string#fragment')).toBe('http://example.com/path');
+  expect(canonicalizeUrl('https://www.example.com/path/to/file.html?param=value')).toBe('https://www.example.com/path/to/file.html');
+  expect(canonicalizeUrl('https://www.example.com/?param=value#fragment')).toBe('https://www.example.com/');
+});
+test('returns <inline> when matching the page URL of the loader', async () => {
+  jest.doMock('../util/global-scope', () => ({
+    initialLocation: 'http://example.com/'
+  }));
+  const {
+    canonicalizeUrl
+  } = await import('./canonicalize-url');
+  expect(canonicalizeUrl('http://example.com/')).toEqual('<inline>');
+});
+test('does not identify sub-paths of the loader origin as <inline>', async () => {
+  jest.doMock('../util/global-scope', () => ({
+    initialLocation: 'http://example.com/'
+  }));
+  const {
+    canonicalizeUrl
+  } = await import('./canonicalize-url');
+  expect(canonicalizeUrl('http://example.com/path/to/script.js')).not.toEqual('<inline>');
+});

package/dist/esm/common/url/clean-url.js

@@ -3,8 +3,15 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-var withHash = /([^?#]*)[^#]*(#[^?]*|$).*/;
-var withoutHash = /([^?#]*)().*/;
+var patternWithHash = /([^?#]*)[^#]*(#[^?]*|$).*/;
+var patternWithoutHash = /([^?#]*)().*/;
+
+/**
+ * Cleans a URL by removing the query string and fragment (hash portion).
+ * @param {string} url - The original URL to be cleaned.
+ * @param {boolean} [keepHash=false] - Whether to preserve the hash portion of the URL.
+ * @returns {string} The cleaned URL.
+ */
 export function cleanURL(url, keepHash) {
-  return url.replace(keepHash ? withHash : withoutHash, '$1$2');
+  return url.replace(keepHash ? patternWithHash : patternWithoutHash, '$1$2');
 }

package/dist/esm/common/util/global-scope.js

@@ -14,6 +14,7 @@ export let globalScope = (() => {
   }
   throw new Error('New Relic browser agent shutting down due to error: Unable to locate global scope. This is possibly due to code redefining browser global variables like "self" and "window".');
 })();
+export const initialLocation = '' + globalScope.location;
 
 /**
  * The below methods are only used for testing and should be removed once the

package/dist/esm/common/wrap/wrap-fetch.js

@@ -7,7 +7,6 @@
  * This module is used by: ajax, spa.
  */
 import { ee as baseEE } from '../event-emitter/contextual-ee';
-import slice from 'lodash._slice';
 import { globalScope } from '../util/global-scope';
 import { flag } from './wrap-function';
 var prefix = 'fetch-';
@@ -68,7 +67,7 @@ export function wrapFetch(sharedEE) {
     var fn = target[name];
     if (typeof fn === 'function') {
       target[name] = function () {
-        var args = slice(arguments);
+        var args = Array.from(arguments);
         var ctx = {};
         // we are wrapping args in an array so we can preserve the reference
         ee.emit(prefix + 'before-start', [args], ctx);