@newrelic/browser-agent 1.261.2 → 1.263.0

package/dist/cjs/loaders/agent.js

@@ -15,7 +15,6 @@ var _instrument = require("../features/page_view_event/instrument");
 var _aggregator = require("../common/aggregate/aggregator");
 var _nreum = require("../common/window/nreum");
 var _console = require("../common/util/console");
-var _stringify = require("../common/util/stringify");
 var _runtime = require("../common/constants/runtime");
 // important side effects
 
@@ -39,7 +38,7 @@ class Agent extends _agentBase.AgentBase {
     if (!_runtime.globalScope) {
       // We could not determine the runtime environment. Short-circuite the agent here
       // to avoid possible exceptions later that may cause issues with customer's application.
-      (0, _console.warn)('Failed to initialize the agent. Could not determine the runtime environment.');
+      (0, _console.warn)(21);
       return;
     }
     this.sharedAggregator = new _aggregator.Aggregator({
@@ -78,11 +77,11 @@ class Agent extends _agentBase.AgentBase {
         if (!this.runSoftNavOverSpa && InstrumentCtor.featureName === _features.FEATURE_NAMES.softNav) return;
         const dependencies = (0, _featureDependencies.getFeatureDependencyNames)(InstrumentCtor.featureName);
         const hasAllDeps = dependencies.every(featName => featName in this.features); // any other feature(s) this depends on should've been initialized on prior iterations by priority order
-        if (!hasAllDeps) (0, _console.warn)("".concat(InstrumentCtor.featureName, " is enabled but one or more dependent features has not been initialized (").concat((0, _stringify.stringify)(dependencies), "). This may cause unintended consequences or missing data..."));
+        if (!hasAllDeps) (0, _console.warn)(36, InstrumentCtor.featureName);
         this.features[InstrumentCtor.featureName] = new InstrumentCtor(this.agentIdentifier, this.sharedAggregator);
       });
     } catch (err) {
-      (0, _console.warn)('Failed to initialize all enabled instrument classes (agent aborted) -', err);
+      (0, _console.warn)(22, err);
       for (const featName in this.features) {
         // this.features hold only features that have been instantiated
         this.features[featName].abortHandler?.();

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

@@ -43,7 +43,9 @@ function setTopLevelCallers() {
     }
     let returnVals = [];
     Object.values(nr.initializedAgents).forEach(val => {
-      if (val.exposed && val.api[fnName]) {
+      if (!val || !val.api) {
+        (0, _console.warn)(38, fnName);
+      } else if (val.exposed && val.api[fnName]) {
         returnVals.push(val.api[fnName](...args));
       }
     });
@@ -68,23 +70,18 @@ function setAPI(agentIdentifier, forceDrain) {
       customAttributes = {},
       level = _constants4.LOG_LEVELS.INFO
     } = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : {};
-    if (!customAttributes || typeof customAttributes !== 'object') customAttributes = {};
-    if (typeof message !== 'string' || !message) return (0, _console.warn)(_constants4.LOGGING_IGNORED + 'invalid message');
-    if (!(0, _utils.isValidLogLevel)(level)) return (0, _console.warn)(_constants4.LOGGING_LEVEL_FAILURE_MESSAGE + level, _constants4.LOG_LEVELS);
-    if (message.length > _constants4.MAX_PAYLOAD_SIZE) return (0, _console.warn)(_constants4.LOGGING_IGNORED + '> ' + _constants4.MAX_PAYLOAD_SIZE + ' bytes: ', message.slice(0, 25) + '...');
-    (0, _utils.bufferLog)(instanceEE, message, customAttributes, level.toUpperCase());
+    (0, _handle.handle)(_constants.SUPPORTABILITY_METRIC_CHANNEL, ['API/log/called'], undefined, _features.FEATURE_NAMES.metrics, instanceEE);
+    (0, _utils.bufferLog)(instanceEE, message, customAttributes, level);
   };
   apiInterface.wrapLogger = function (parent, functionName) {
     let {
       customAttributes = {},
       level = _constants4.LOG_LEVELS.INFO
     } = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : {};
-    if (!customAttributes || typeof customAttributes !== 'object') customAttributes = {};
-    if (!(typeof parent === 'object' && !!parent && typeof functionName === 'string' && !!functionName && typeof parent[functionName] === 'function' && typeof customAttributes === 'object')) return (0, _console.warn)(_constants4.LOGGING_FAILURE_MESSAGE + 'invalid argument(s)');
-    if (!(0, _utils.isValidLogLevel)(level)) return (0, _console.warn)(_constants4.LOGGING_FAILURE_MESSAGE + _constants4.LOGGING_LEVEL_FAILURE_MESSAGE + level, _constants4.LOG_LEVELS);
+    (0, _handle.handle)(_constants.SUPPORTABILITY_METRIC_CHANNEL, ['API/wrapLogger/called'], undefined, _features.FEATURE_NAMES.metrics, instanceEE);
     (0, _wrapLogger.wrapLogger)(instanceEE, parent, functionName, {
       customAttributes,
-      level: level.toUpperCase()
+      level
     });
   };
 
@@ -126,11 +123,11 @@ function setAPI(agentIdentifier, forceDrain) {
   apiInterface.setCustomAttribute = function (name, value) {
     let persistAttribute = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : false;
     if (typeof name !== 'string') {
-      (0, _console.warn)("Failed to execute setCustomAttribute.\nName must be a string type, but a type of <".concat(typeof name, "> was provided."));
+      (0, _console.warn)(39, typeof name);
       return;
     }
     if (!(['string', 'number', 'boolean'].includes(typeof value) || value === null)) {
-      (0, _console.warn)("Failed to execute setCustomAttribute.\nNon-null value must be a string, number or boolean type, but a type of <".concat(typeof value, "> was provided."));
+      (0, _console.warn)(40, typeof value);
       return;
     }
     return appendJsAttribute(name, value, 'setCustomAttribute', persistAttribute);
@@ -142,7 +139,7 @@ function setAPI(agentIdentifier, forceDrain) {
    */
   apiInterface.setUserId = function (value) {
     if (!(typeof value === 'string' || value === null)) {
-      (0, _console.warn)("Failed to execute setUserId.\nNon-null value must be a string type, but a type of <".concat(typeof value, "> was provided."));
+      (0, _console.warn)(41, typeof value);
       return;
     }
     return appendJsAttribute('enduser.id', value, 'setUserId', true);
@@ -155,7 +152,7 @@ function setAPI(agentIdentifier, forceDrain) {
    */
   apiInterface.setApplicationVersion = function (value) {
     if (!(typeof value === 'string' || value === null)) {
-      (0, _console.warn)("Failed to execute setApplicationVersion. Expected <String | null>, but got <".concat(typeof value, ">."));
+      (0, _console.warn)(42, typeof value);
       return;
     }
     return appendJsAttribute('application.version', value, 'setApplicationVersion', false);
@@ -165,7 +162,7 @@ function setAPI(agentIdentifier, forceDrain) {
       (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);
+      (0, _console.warn)(23, err);
     }
   };
   apiInterface[_constants2.SR_EVENT_EMITTER_TYPES.RECORD] = function () {
@@ -234,7 +231,7 @@ function setAPI(agentIdentifier, forceDrain) {
       setAPI(agentIdentifier);
       (0, _drain.drain)(agentIdentifier, 'api');
     }).catch(err => {
-      (0, _console.warn)('Downloading runtime APIs failed...', err);
+      (0, _console.warn)(27, err);
       instanceEE.abort();
     });
   }

package/dist/cjs/loaders/micro-agent.js

@@ -65,11 +65,11 @@ class MicroAgent extends _agentBase.AgentBase {
       const featNames = nonAutoFeatures;
       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.some(f => !featNames.includes(f))) return (0, _console.warn)(37, featNames);
         if (!features.includes(_features.FEATURE_NAMES.pageViewEvent)) features.push(_features.FEATURE_NAMES.pageViewEvent);
       }
     } catch (err) {
-      (0, _console.warn)('An unexpected issue occurred', err);
+      (0, _console.warn)(23, err);
     }
     try {
       const enabledFeatures = (0, _enabledFeatures.getEnabledFeatures)(this.agentIdentifier);
@@ -77,7 +77,7 @@ class MicroAgent extends _agentBase.AgentBase {
         // a biproduct of doing this is that the "session manager" is automatically handled through importing this feature
         this.features.page_view_event = new _instrument.Instrument(this.agentIdentifier, this.sharedAggregator);
       } catch (err) {
-        (0, _console.warn)('Something prevented the agent from instrumenting.', err);
+        (0, _console.warn)(24, err);
       }
       (0, _load.onWindowLoad)(() => {
         // these features do not import an "instrument" file, meaning they are only hooked up to the API.
@@ -93,13 +93,13 @@ class MicroAgent extends _agentBase.AgentBase {
                 Aggregate
               } = _ref2;
               this.features[f] = new Aggregate(this.agentIdentifier, this.sharedAggregator);
-            }).catch(err => (0, _console.warn)('Something prevented the agent from being downloaded.', err));
+            }).catch(err => (0, _console.warn)(25, err));
           }
         });
       });
       return true;
     } catch (err) {
-      (0, _console.warn)('Failed to initialize instrument classes.', err);
+      (0, _console.warn)(26, err);
       return false;
     }
   }

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

@@ -1,8 +1,8 @@
 import { warn } from '../../util/console';
 export function getModeledObject(obj, model) {
   try {
-    if (!obj || typeof obj !== 'object') return warn('Setting a Configurable requires an object as input');
-    if (!model || typeof model !== 'object') return warn('Setting a Configurable requires a model to set its initial properties');
+    if (!obj || typeof obj !== 'object') return warn(3);
+    if (!model || typeof model !== 'object') return warn(4);
     // allow getters and setters to pass from model to target
     const output = Object.create(Object.getPrototypeOf(model), Object.getOwnPropertyDescriptors(model));
     const target = Object.keys(output).length === 0 ? obj : output;
@@ -15,11 +15,11 @@ export function getModeledObject(obj, model) {
         }
         if (Array.isArray(obj[key]) && Array.isArray(model[key])) output[key] = Array.from(new Set([...obj[key], ...model[key]]));else if (typeof obj[key] === 'object' && typeof model[key] === 'object') output[key] = getModeledObject(obj[key], model[key]);else output[key] = obj[key];
       } catch (e) {
-        warn('An error occurred while setting a property of a Configurable', e);
+        warn(1, e);
       }
     }
     return output;
   } catch (err) {
-    warn('An error occured while setting a Configurable', err);
+    warn(2, err);
   }
 }

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

@@ -109,6 +109,8 @@ const model = () => {
       // 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
+      fix_stylesheets: true,
+      // fetch missing stylesheet resources for inlining, only works if 'inline_stylesheet' is also true
       // recording config settings
       mask_all_inputs: true,
       // this has a getter/setter to facilitate validation of the selectors
@@ -116,7 +118,7 @@ const model = () => {
         return hiddenState.mask_selector;
       },
       set mask_text_selector(val) {
-        if (isValidSelector(val)) hiddenState.mask_selector = "".concat(val, ",").concat(nrMask);else if (val === '' || val === null) hiddenState.mask_selector = nrMask;else warn('An invalid session_replay.mask_selector was provided. \'*\' will be used.', val);
+        if (isValidSelector(val)) hiddenState.mask_selector = "".concat(val, ",").concat(nrMask);else if (val === '' || val === null) hiddenState.mask_selector = nrMask;else warn(5, val);
       },
       // these properties only have getters because they are enforcable constants and should error if someone tries to override them
       get block_class() {
@@ -134,7 +136,7 @@ const model = () => {
         return hiddenState.block_selector;
       },
       set block_selector(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);
+        if (isValidSelector(val)) hiddenState.block_selector += ",".concat(val);else if (val !== '') warn(6, val);
       },
       // password: must always be present and true no matter what customer sets
       get mask_input_options() {
@@ -144,7 +146,7 @@ const model = () => {
         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);
+        };else warn(7, val);
       }
     },
     session_trace: {

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

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

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

@@ -6,7 +6,7 @@ const model = {
 export class SharedContext {
   constructor(context) {
     try {
-      if (typeof context !== 'object') return warn('shared context requires an object as input');
+      if (typeof context !== 'object') return warn(8);
       this.sharedContext = {};
       Object.assign(this.sharedContext, model);
       Object.entries(context).forEach(_ref => {
@@ -14,7 +14,7 @@ export class SharedContext {
         if (Object.keys(model).includes(key)) this.sharedContext[key] = value;
       });
     } catch (err) {
-      warn('An error occurred while setting SharedContext', err);
+      warn(9, err);
     }
   }
 }

package/dist/esm/common/drain/drain.js

@@ -34,8 +34,9 @@ export function registerDrain(agentIdentifier, group) {
  * @param {*} group - The named "bucket" to be removed from the registry
  */
 export function deregisterDrain(agentIdentifier, group) {
-  curateRegistry(agentIdentifier);
+  if (!agentIdentifier || !registry[agentIdentifier]) return;
   if (registry[agentIdentifier].get(group)) registry[agentIdentifier].delete(group);
+  drainGroup(agentIdentifier, group, false);
   if (registry[agentIdentifier].size) checkCanDrainAll(agentIdentifier);
 }
 
@@ -95,27 +96,32 @@ function checkCanDrainAll(agentIdentifier) {
    * @param {*} group - The name of a particular feature's event "bucket".
    */
 function drainGroup(agentIdentifier, group) {
+  let activateGroup = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : true;
   const baseEE = agentIdentifier ? ee.get(agentIdentifier) : ee;
   const handlers = defaultRegister.handlers; // other storage in registerHandler
-  if (!baseEE.backlog || !handlers) return;
-  var bufferedEventsInGroup = baseEE.backlog[group];
-  var groupHandlers = handlers[group]; // each group in the registerHandler storage
-  if (groupHandlers) {
-    // We don't cache the length of the buffer while looping because events might still be added while processing.
-    for (var i = 0; bufferedEventsInGroup && i < bufferedEventsInGroup.length; ++i) {
-      // eslint-disable-line no-unmodified-loop-condition
-      emitEvent(bufferedEventsInGroup[i], groupHandlers);
-    }
-    mapOwn(groupHandlers, function (eventType, handlerRegistrationList) {
-      mapOwn(handlerRegistrationList, function (i, registration) {
-        // registration is an array of: [targetEE, eventHandler]
-        registration[0].on(eventType, registration[1]);
+  if (baseEE.aborted || !baseEE.backlog || !handlers) return;
+
+  // Only activated features being drained should run queued listeners on buffered events. Deactivated features only need to release memory.
+  if (activateGroup) {
+    const bufferedEventsInGroup = baseEE.backlog[group];
+    const groupHandlers = handlers[group]; // each group in the registerHandler storage
+    if (groupHandlers) {
+      // We don't cache the length of the buffer while looping because events might still be added while processing.
+      for (let i = 0; bufferedEventsInGroup && i < bufferedEventsInGroup.length; ++i) {
+        // eslint-disable-line no-unmodified-loop-condition
+        emitEvent(bufferedEventsInGroup[i], groupHandlers);
+      }
+      mapOwn(groupHandlers, function (eventType, handlerRegistrationList) {
+        mapOwn(handlerRegistrationList, function (i, registration) {
+          // registration is an array of: [targetEE, eventHandler]
+          registration[0].on(eventType, registration[1]);
+        });
       });
-    });
+    }
   }
   if (!baseEE.isolatedBacklog) delete handlers[group];
   baseEE.backlog[group] = null;
-  baseEE.emit('drain-' + group, []);
+  baseEE.emit('drain-' + group, []); // TODO: Code exists purely for a unit test and needs to be refined
 }
 
 /**

package/dist/esm/common/harvest/harvest.js

@@ -148,7 +148,7 @@ export class Harvest extends SharedContext {
         body = stringify(body);
       }
       /** Warn --once per endpoint-- if the agent tries to send large payloads */
-      if (body.length > 750000 && (warnings[endpoint] = (warnings?.[endpoint] || 0) + 1) === 1) warn("The Browser Agent is attempting to send a very large payload to /".concat(endpoint, ". This is usually tied to large amounts of custom attributes. Please check your configurations."));
+      if (body.length > 750000 && (warnings[endpoint] = (warnings?.[endpoint] || 0) + 1) === 1) warn(28, endpoint);
     }
     if (!body || body.length === 0 || body === '{}' || body === '[]') {
       // If body is null, undefined, or an empty object or array, send an empty string instead

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

@@ -190,7 +190,7 @@ export class SessionEntity {
       }
       return obj;
     } catch (e) {
-      warn('Failed to read from storage API', e);
+      warn(10, e);
       // storage is inaccessible
       return {};
     }
@@ -216,7 +216,7 @@ export class SessionEntity {
       return data;
     } catch (e) {
       // storage is inaccessible
-      warn('Failed to write to the storage API', e);
+      warn(11, e);
       return null;
     }
   }

package/dist/esm/common/timing/time-keeper.js

@@ -1,5 +1,6 @@
 import { originTime } from '../constants/runtime';
 import { getRuntime } from '../config/config';
+const rfc2616Regex = /^(Mon|Tue|Wed|Thu|Fri|Sat|Sun), ([0-3][0-9]) (Jan|Feb|Mar|Apr|May|Jun|Jul|Aug|Sep|Oct|Nov|Dec) ([0-9]{4}) ([01][0-9]|2[0-3])(:[0-5][0-9]){2} GMT$/;
 
 /**
  * Class used to adjust the timestamp of harvested data to New Relic server time. This
@@ -29,7 +30,7 @@ export class TimeKeeper {
   /**
    * Represents whether the timekeeper is in a state that it can accurately convert
    * timestamps.
-   * @type {number}
+   * @type {boolean}
    */
   #ready = false;
   constructor(agentIdentifier) {
@@ -42,6 +43,9 @@ export class TimeKeeper {
   get correctedOriginTime() {
     return this.#correctedOriginTime;
   }
+  get localTimeDiff() {
+    return this.#localTimeDiff;
+  }
 
   /**
    * Process a rum request to calculate NR server time.
@@ -50,12 +54,16 @@ export class TimeKeeper {
    * @param endTime {number} The end time of the RUM request
    */
   processRumRequest(rumRequest, startTime, endTime) {
-    this.processStoredDiff();
+    this.processStoredDiff(); // Check session entity for stored time diff
     if (this.#ready) return; // Server time calculated from session entity
+
     const responseDateHeader = rumRequest.getResponseHeader('Date');
     if (!responseDateHeader) {
       throw new Error('Missing date header on rum response.');
     }
+    if (!rfc2616Regex.test(responseDateHeader)) {
+      throw new Error('Date header invalid format.');
+    }
     const medianRumOffset = (endTime - startTime) / 2;
     const serverOffset = startTime + medianRumOffset;
 
@@ -92,6 +100,8 @@ export class TimeKeeper {
 
   /** Process the session entity and use the info to set the main time calculations if present */
   processStoredDiff() {
+    if (this.#ready) return; // Time diff has already been calculated
+
     const storedServerTimeDiff = this.#session?.read()?.serverTimeDiff;
     if (typeof storedServerTimeDiff === 'number' && !isNaN(storedServerTimeDiff)) {
       this.#localTimeDiff = storedServerTimeDiff;

package/dist/esm/common/util/console.js

@@ -4,8 +4,7 @@
  * @param {*} [secondary] Secondary data to include, usually an error or object
  * @returns
  */
-export function warn(message, secondary) {
-  if (typeof console.warn !== 'function') return;
-  console.warn("New Relic: ".concat(message));
-  if (secondary) console.warn(secondary);
+export function warn(code, secondary) {
+  if (typeof console.debug !== 'function') return;
+  console.debug("New Relic Warning: https://github.com/newrelic/newrelic-browser-agent/blob/main/docs/warning-codes.md#".concat(code), secondary);
 }

package/dist/esm/common/util/obfuscate.js

@@ -45,15 +45,15 @@ export function validateRules(rules) {
   var invalidRegexDetected = false;
   for (var i = 0; i < rules.length; i++) {
     if (!('regex' in rules[i])) {
-      warn('An obfuscation replacement rule was detected missing a "regex" value.');
+      warn(12);
       invalidRegexDetected = true;
     } else if (typeof rules[i].regex !== 'string' && !(rules[i].regex instanceof RegExp)) {
-      warn('An obfuscation replacement rule contains a "regex" value with an invalid type (must be a string or RegExp)');
+      warn(13);
       invalidRegexDetected = true;
     }
     var replacement = rules[i].replacement;
     if (replacement && typeof replacement !== 'string') {
-      warn('An obfuscation replacement rule contains a "replacement" value with an invalid type (must be a string)');
+      warn(14);
       invalidReplacementDetected = true;
     }
   }

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

@@ -9,6 +9,7 @@
 
 import { ee as baseEE, contextId } from '../event-emitter/contextual-ee';
 import { EventContext } from '../event-emitter/event-context';
+import { warn } from '../util/console';
 import { createWrapperWithEmitter as wfn } from './wrap-function';
 
 /**
@@ -20,6 +21,7 @@ import { createWrapperWithEmitter as wfn } from './wrap-function';
  */
 // eslint-disable-next-line
 export function wrapLogger(sharedEE, parent, loggerFn, context) {
+  if (!(typeof parent === 'object' && !!parent && typeof loggerFn === 'string' && !!loggerFn && typeof parent[loggerFn] === 'function')) return warn(29);
   const ee = scopedEE(sharedEE);
   const wrapFn = wfn(ee);
 

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

@@ -49,7 +49,7 @@ export function wrapXhr(sharedEE) {
       ee.emit('new-xhr', [xhr], context);
       xhr.addEventListener(READY_STATE_CHANGE, wrapXHR(context), eventListenerOpts(false));
     } catch (e) {
-      warn('An error occurred while intercepting XHR', e);
+      warn(15, e);
       try {
         ee.emit('internal-error', [e]);
       } catch (err) {

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

@@ -6,8 +6,9 @@ import { warn } from '../../../common/util/console';
 import { stringify } from '../../../common/util/stringify';
 import { SUPPORTABILITY_METRIC_CHANNEL } from '../../metrics/constants';
 import { AggregateBase } from '../../utils/aggregate-base';
-import { FEATURE_NAME, LOGGING_EVENT_EMITTER_CHANNEL, LOGGING_IGNORED, MAX_PAYLOAD_SIZE } from '../constants';
+import { FEATURE_NAME, LOGGING_EVENT_EMITTER_CHANNEL, LOG_LEVELS, MAX_PAYLOAD_SIZE } from '../constants';
 import { Log } from '../shared/log';
+import { isValidLogLevel } from '../shared/utils';
 export class Aggregate extends AggregateBase {
   static featureName = FEATURE_NAME;
   #agentRuntime;
@@ -38,13 +39,37 @@ export class Aggregate extends AggregateBase {
       this.drain();
     });
   }
-  handleLog(timestamp, message, attributes, level) {
+  handleLog(timestamp, message) {
+    let attributes = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : {};
+    let level = arguments.length > 3 && arguments[3] !== undefined ? arguments[3] : LOG_LEVELS.INFO;
     if (this.blocked) return;
+    if (!attributes || typeof attributes !== 'object') attributes = {};
+    if (typeof level === 'string') level = level.toUpperCase();
+    if (!isValidLogLevel(level)) return warn(30, level);
+    try {
+      if (typeof message !== 'string') {
+        const stringified = stringify(message);
+        /**
+           * Error instances convert to `{}` when stringified
+           * Symbol converts to '' when stringified
+           * other cases tbd
+           * */
+        if (!!stringified && stringified !== '{}') message = stringified;else message = String(message);
+      }
+    } catch (err) {
+      warn(16, message);
+      return;
+    }
+    if (typeof message !== 'string' || !message) return warn(32);
+    if (message.length > MAX_PAYLOAD_SIZE) {
+      handle(SUPPORTABILITY_METRIC_CHANNEL, ['Logging/Harvest/Failed/Seen', message.length]);
+      return warn(31, message.slice(0, 25) + '...');
+    }
     const log = new Log(this.#agentRuntime.timeKeeper.convertRelativeTimestamp(timestamp), message, attributes, level);
     const logBytes = log.message.length + stringify(log.attributes).length + log.level.length + 10; // timestamp == 10 chars
     if (logBytes > MAX_PAYLOAD_SIZE) {
       handle(SUPPORTABILITY_METRIC_CHANNEL, ['Logging/Harvest/Failed/Seen', logBytes]);
-      return warn(LOGGING_IGNORED + '> ' + MAX_PAYLOAD_SIZE + ' bytes', log.message.slice(0, 25) + '...');
+      return warn(31, log.message.slice(0, 25) + '...');
     }
     if (this.estimatedBytes + logBytes >= MAX_PAYLOAD_SIZE) {
       handle(SUPPORTABILITY_METRIC_CHANNEL, ['Logging/Harvest/Early/Seen', this.estimatedBytes + logBytes]);

package/dist/esm/features/logging/constants.js

@@ -8,7 +8,4 @@ export const LOG_LEVELS = {
 };
 export const LOGGING_EVENT_EMITTER_CHANNEL = 'log';
 export const FEATURE_NAME = FEATURE_NAMES.logging;
-export const MAX_PAYLOAD_SIZE = 1000000;
-export const LOGGING_FAILURE_MESSAGE = 'failed to wrap logger: ';
-export const LOGGING_LEVEL_FAILURE_MESSAGE = 'invalid log level: ';
-export const LOGGING_IGNORED = 'ignored log: ';
+export const MAX_PAYLOAD_SIZE = 1000000;

package/dist/esm/features/logging/shared/utils.js

@@ -1,7 +1,5 @@
 import { handle } from '../../../common/event-emitter/handle';
 import { now } from '../../../common/timing/now';
-import { warn } from '../../../common/util/console';
-import { stringify } from '../../../common/util/stringify';
 import { FEATURE_NAMES } from '../../../loaders/features/features';
 import { SUPPORTABILITY_METRIC_CHANNEL } from '../../metrics/constants';
 import { LOGGING_EVENT_EMITTER_CHANNEL, LOG_LEVELS } from '../constants';
@@ -15,30 +13,16 @@ import { LOGGING_EVENT_EMITTER_CHANNEL, LOG_LEVELS } from '../constants';
 export function bufferLog(ee, message) {
   let customAttributes = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : {};
   let level = arguments.length > 3 && arguments[3] !== undefined ? arguments[3] : LOG_LEVELS.INFO;
-  try {
-    if (typeof message !== 'string') {
-      const stringified = stringify(message);
-      /**
-       * Error instances convert to `{}` when stringified
-       * Symbol converts to '' when stringified
-       * other cases tbd
-       * */
-      if (!!stringified && stringified !== '{}') message = stringified;else message = String(message);
-    }
-  } catch (err) {
-    warn('could not cast log message to string', message);
-    return;
-  }
   handle(SUPPORTABILITY_METRIC_CHANNEL, ["API/logging/".concat(level.toLowerCase(), "/called")], undefined, FEATURE_NAMES.metrics, ee);
   handle(LOGGING_EVENT_EMITTER_CHANNEL, [now(), message, customAttributes, level], undefined, FEATURE_NAMES.logging, ee);
 }
 
 /**
  * Checks if a supplied log level is acceptable for use in generating a log event
- * @param {string} level
+ * @param {string} level -- must be cast to uppercase before running this test
  * @returns {boolean}
  */
 export function isValidLogLevel(level) {
   if (typeof level !== 'string') return false;
-  return Object.values(LOG_LEVELS).some(logLevel => logLevel.toUpperCase() === level.toUpperCase());
+  return Object.values(LOG_LEVELS).some(logLevel => logLevel === level);
 }

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

@@ -102,6 +102,22 @@ export class Aggregate extends AggregateBase {
     // Check if proxy for either chunks or beacon is being used
     if (proxy.assets) this.storeSupportabilityMetrics('Config/AssetsUrl/Changed');
     if (proxy.beacon) this.storeSupportabilityMetrics('Config/BeaconUrl/Changed');
+    if (isBrowserScope && window.MutationObserver) {
+      this.storeSupportabilityMetrics('Generic/VideoElement/Added', window.document.querySelectorAll('video').length);
+      const mo = new MutationObserver(records => {
+        records.forEach(record => {
+          record.addedNodes.forEach(addedNode => {
+            if (addedNode instanceof HTMLVideoElement) {
+              this.storeSupportabilityMetrics('Generic/VideoElement/Added', 1);
+            }
+          });
+        });
+      });
+      mo.observe(window.document.body, {
+        childList: true,
+        subtree: true
+      });
+    }
   }
   eachSessionChecks() {
     if (!isBrowserScope) return;