@microsoft/applicationinsights-core-js 2.8.0-nightly.2204-17 → 2.8.0

package/src/JavaScriptSDK/BaseCore.ts

@@ -20,7 +20,10 @@ import { IPerfManager } from "../JavaScriptSDK.Interfaces/IPerfManager";
 import { getGblPerfMgr, PerfManager } from "./PerfManager";
 import { ICookieMgr } from "../JavaScriptSDK.Interfaces/ICookieMgr";
 import { createCookieMgr } from "./CookieMgr";
-import { arrForEach, isNullOrUndefined, toISOString, getSetValue, setValue, throwError, isNotTruthy, isFunction, objFreeze, proxyFunctionAs, proxyFunctions } from "./HelperFuncs";
+import {
+    arrForEach, isNullOrUndefined, getSetValue, setValue, isNotTruthy, isFunction, objExtend, objFreeze, proxyFunctionAs, proxyFunctions, throwError,
+    toISOString
+} from "./HelperFuncs";
 import { strExtensionConfig, strIKey } from "./Constants";
 import { DiagnosticLogger, _InternalLogMessage, _throwInternal, _warnToConsole } from "./DiagnosticLogger";
 import { getDebugListener } from "./DbgExtensionUtils";
@@ -43,6 +46,11 @@ const strSdkUnloadingError = "SDK is still unloading...";
 const strSdkNotInitialized = "SDK is not initialized";
 // const strPluginUnloadFailed = "Failed to unload plugin";
 
+const defaultInitConfig = {
+    // Have the Diagnostic Logger default to log critical errors to the console
+    loggingLevelConsole: eLoggingSeverity.CRITICAL
+};
+
 /**
  * Helper to create the default performance manager
  * @param core
@@ -361,7 +369,7 @@ export class BaseCore implements IAppInsightsCore {
                     flushComplete: false
                 }
 
-                let processUnloadCtx = createProcessTelemetryUnloadContext(_getPluginChain(), _self.config, _self);
+                let processUnloadCtx = createProcessTelemetryUnloadContext(_getPluginChain(), _self);
                 processUnloadCtx.onComplete(() => {
                     _initDefaults();
                     unloadComplete && unloadComplete(unloadState);
@@ -451,8 +459,8 @@ export class BaseCore implements IAppInsightsCore {
                 _isInitialized = false;
 
                 // Use a default logger so initialization errors are not dropped on the floor with full logging
-                _self.logger = new DiagnosticLogger({ loggingLevelConsole: eLoggingSeverity.CRITICAL });
-                _self.config = {};
+                _self.config = objExtend(true, {}, defaultInitConfig);
+                _self.logger = new DiagnosticLogger(_self.config);
                 _self._extensions = [];
 
                 _telemetryInitializerPlugin = new TelemetryInitializerPlugin();
@@ -601,7 +609,7 @@ export class BaseCore implements IAppInsightsCore {
 
                 if (thePlugins && thePlugins.length > 0) {
                     let unloadChain = createTelemetryProxyChain(thePlugins, _self.config, _self);
-                    let unloadCtx = createProcessTelemetryUnloadContext(unloadChain, _self.config, _self);
+                    let unloadCtx = createProcessTelemetryUnloadContext(unloadChain, _self);
 
                     unloadCtx.onComplete(() => {
                         let removed = false;
@@ -705,7 +713,7 @@ export class BaseCore implements IAppInsightsCore {
             }
 
             function _doUpdate(updateState: ITelemetryUpdateState): void {
-                let updateCtx = createProcessTelemetryUpdateContext(_getPluginChain(), _self.config, _self);
+                let updateCtx = createProcessTelemetryUpdateContext(_getPluginChain(), _self);
 
                 if (!_self._updateHook || _self._updateHook(updateCtx, updateState) !== true) {
                     updateCtx.processNext(updateState);

package/src/JavaScriptSDK/BaseTelemetryPlugin.ts

@@ -120,14 +120,15 @@ export abstract class BaseTelemetryPlugin implements ITelemetryPlugin {
             _self.teardown = (unloadCtx?: IProcessTelemetryUnloadContext, unloadState?: ITelemetryUnloadState) => {
                 // If this plugin has already been torn down (not operational) or is not initialized (core is not set)
                 // or the core being used for unload was not the same core used for initialization.
-                if (!_self.core || (unloadCtx && _self.core !== unloadCtx.core())) {
+                let core = _self.core;
+                if (!core || (unloadCtx && core !== unloadCtx.core())) {
                     // Do Nothing as either the plugin is not initialized or was not initialized by the current core
                     return;
                 }
 
                 let result: void | boolean;
                 let unloadDone = false;
-                let theUnloadCtx = unloadCtx || createProcessTelemetryUnloadContext(null, {}, _self.core, _nextPlugin && _nextPlugin[strGetPlugin] ? _nextPlugin[strGetPlugin]() : _nextPlugin);
+                let theUnloadCtx = unloadCtx || createProcessTelemetryUnloadContext(null, core, _nextPlugin && _nextPlugin[strGetPlugin] ? _nextPlugin[strGetPlugin]() : _nextPlugin);
                 let theUnloadState: ITelemetryUnloadState = unloadState || {
                     reason: TelemetryUnloadReason.ManualTeardown,
                     isAsync: false
@@ -166,14 +167,15 @@ export abstract class BaseTelemetryPlugin implements ITelemetryPlugin {
             _self.update = (updateCtx: IProcessTelemetryUpdateContext, updateState: ITelemetryUpdateState) => {
                 // If this plugin has already been torn down (not operational) or is not initialized (core is not set)
                 // or the core being used for unload was not the same core used for initialization.
-                if (!_self.core || (updateCtx && _self.core !== updateCtx.core())) {
+                let core = _self.core;
+                if (!core || (updateCtx && core !== updateCtx.core())) {
                     // Do Nothing
                     return;
                 }
 
                 let result: void | boolean;
                 let updateDone = false;
-                let theUpdateCtx = updateCtx || createProcessTelemetryUpdateContext(null, {}, _self.core, _nextPlugin && _nextPlugin[strGetPlugin] ? _nextPlugin[strGetPlugin]() : _nextPlugin);
+                let theUpdateCtx = updateCtx || createProcessTelemetryUpdateContext(null, core, _nextPlugin && _nextPlugin[strGetPlugin] ? _nextPlugin[strGetPlugin]() : _nextPlugin);
                 let theUpdateState: ITelemetryUpdateState = updateState || {
                     reason: TelemetryUpdateReason.Unknown
                 };

package/src/JavaScriptSDK/DataCacheHelper.ts

@@ -7,7 +7,7 @@ import { newId } from "./RandomHelper";
 
 const _objDefineProperty = ObjDefineProperty;
 
-const version = "2.8.0-nightly.2204-17";
+const version = "2.8.0";
 let instanceName = "." + newId(6);
 let _dataUid = 0;
 

package/src/JavaScriptSDK/ProcessTelemetryContext.ts

@@ -43,7 +43,7 @@ interface IInternalContext<T extends IBaseProcessingContext> {
     ctx: T
 }
 
-function _getNextProxyStart(proxy: ITelemetryPluginChain, config: IConfiguration, core:IAppInsightsCore, startAt: IPlugin): ITelemetryPluginChain {
+function _getNextProxyStart<T, C = IConfiguration>(proxy: ITelemetryPluginChain, core: IAppInsightsCore, startAt: IPlugin): ITelemetryPluginChain {
     while (proxy) {
         if (proxy.getPlugin() === startAt) {
             return proxy;
@@ -53,7 +53,7 @@ function _getNextProxyStart(proxy: ITelemetryPluginChain, config: IConfiguration
     }
 
     // This wasn't found in the existing chain so create an isolated one with just this plugin
-    return createTelemetryProxyChain([startAt], config, core);
+    return createTelemetryProxyChain([startAt], core.config || {}, core);
 }
 
 /**
@@ -64,7 +64,7 @@ function _getNextProxyStart(proxy: ITelemetryPluginChain, config: IConfiguration
  * @param startAt - Identifies the next plugin to execute, if null there is no "next" plugin and if undefined it should assume the start of the chain
  * @returns
  */
-function _createInternalContext<T extends IBaseProcessingContext>(telemetryChain: ITelemetryPluginChain, config: IConfiguration, core:IAppInsightsCore, startAt?: IPlugin): IInternalContext<T> {
+function _createInternalContext<T extends IBaseProcessingContext>(telemetryChain: ITelemetryPluginChain, config: IConfiguration, core: IAppInsightsCore, startAt?: IPlugin): IInternalContext<T> {
     // We have a special case where we want to start execution from this specific plugin
     // or we simply reuse the existing telemetry plugin chain (normal execution case)
     let _nextProxy: ITelemetryPluginChain | null = null;  // By Default set as no next plugin
@@ -72,7 +72,7 @@ function _createInternalContext<T extends IBaseProcessingContext>(telemetryChain
 
     if (startAt !== null) {
         // There is no next element (null) vs not defined (undefined) so use the full chain
-        _nextProxy = startAt ? _getNextProxyStart(telemetryChain, config, core, startAt) : telemetryChain;
+        _nextProxy = startAt ? _getNextProxyStart(telemetryChain, core, startAt) : telemetryChain;
     }
 
     let context: IInternalContext<T> = {
@@ -159,7 +159,7 @@ function _createInternalContext<T extends IBaseProcessingContext>(telemetryChain
                 let newConfig = objExtend(true, defaultValue, theConfig);
 
                 if (config && mergeDefault === GetExtCfgMergeType.MergeDefaultFromRootOrDefault) {
-                    // Enumerate over the defaultValues and if not already populate attempt to
+                    // Enumerate over the defaultValues and if not already populated attempt to
                     // find a value from the root config
                     objForEachKey(defaultValue, (field) => {
                         // for each unspecified field, set the default value
@@ -171,6 +171,8 @@ function _createInternalContext<T extends IBaseProcessingContext>(telemetryChain
                         }
                     });
                 }
+
+                theConfig = newConfig;
             }
         }
 
@@ -244,7 +246,8 @@ export function createProcessTelemetryContext(telemetryChain: ITelemetryPluginCh
  * @param core - The current core instance
  * @param startAt - Identifies the next plugin to execute, if null there is no "next" plugin and if undefined it should assume the start of the chain
  */
-export function createProcessTelemetryUnloadContext(telemetryChain: ITelemetryPluginChain, config: IConfiguration, core:IAppInsightsCore, startAt?: IPlugin): IProcessTelemetryUnloadContext {
+export function createProcessTelemetryUnloadContext(telemetryChain: ITelemetryPluginChain, core: IAppInsightsCore, startAt?: IPlugin): IProcessTelemetryUnloadContext {
+    let config = core.config || {};
     let internalContext: IInternalContext<IProcessTelemetryUnloadContext> = _createInternalContext<IProcessTelemetryUnloadContext>(telemetryChain, config, core, startAt);
     let context = internalContext.ctx;
 
@@ -260,7 +263,7 @@ export function createProcessTelemetryUnloadContext(telemetryChain: ITelemetryPl
             plugins = createTelemetryProxyChain(plugins, config, core, startAt);
         }
 
-        return createProcessTelemetryUnloadContext(plugins || context.getNext(), config, core, startAt);
+        return createProcessTelemetryUnloadContext(plugins || context.getNext(), core, startAt);
     }
 
     context.processNext = _processNext;
@@ -276,14 +279,15 @@ export function createProcessTelemetryUnloadContext(telemetryChain: ITelemetryPl
  * @param core - The current core instance
  * @param startAt - Identifies the next plugin to execute, if null there is no "next" plugin and if undefined it should assume the start of the chain
  */
-export function createProcessTelemetryUpdateContext(telemetryChain: ITelemetryPluginChain, config: IConfiguration, core:IAppInsightsCore, startAt?: IPlugin): IProcessTelemetryUpdateContext {
+export function createProcessTelemetryUpdateContext(telemetryChain: ITelemetryPluginChain, core: IAppInsightsCore, startAt?: IPlugin): IProcessTelemetryUpdateContext {
+    let config = core.config || {};
     let internalContext: IInternalContext<IProcessTelemetryUpdateContext> = _createInternalContext<IProcessTelemetryUpdateContext>(telemetryChain, config, core, startAt);
     let context = internalContext.ctx;
 
     function _processNext(updateState: ITelemetryUpdateState) {
         return context.iterate((plugin) => {
-            if (isFunction(plugin[strUpdate])) {
-                plugin[strUpdate](context, updateState);
+            if (isFunction(plugin.update)) {
+                plugin.update(context, updateState);
             }
         });
     }
@@ -293,7 +297,7 @@ export function createProcessTelemetryUpdateContext(telemetryChain: ITelemetryPl
             plugins = createTelemetryProxyChain(plugins, config, core, startAt);
         }
 
-        return createProcessTelemetryUpdateContext(plugins || context.getNext(), config, core, startAt);
+        return createProcessTelemetryUpdateContext(plugins || context.getNext(), core, startAt);
     }
 
     context.processNext = _processNext;
@@ -446,7 +450,7 @@ export function createTelemetryPluginProxy(plugin: ITelemetryPlugin, config: ICo
                             itemCtx.diagLog(),
                             eLoggingSeverity.CRITICAL,
                             _eInternalMessageId.PluginException,
-                            "Plugin [" + plugin.identifier + "] failed during " + name + " - " + dumpObj(error) + ", run flags: " + dumpObj(hasRunContext));
+                            "Plugin [" + identifier + "] failed during " + name + " - " + dumpObj(error) + ", run flags: " + dumpObj(hasRunContext));
                     }
                 }
             }, details, isAsync);
@@ -464,7 +468,7 @@ export function createTelemetryPluginProxy(plugin: ITelemetryPlugin, config: ICo
             }
 
             let pluginState = _getPluginState(plugin);
-            if (pluginState[strTeardown] || pluginState[strDisabled]) {
+            if (pluginState.teardown || pluginState[strDisabled]) {
                 return false;
             }
 
@@ -627,7 +631,7 @@ export class ProcessTelemetryContext implements IProcessTelemetryContext {
      * @param config - The current config
      * @param core - The current core instance
      */
-    constructor(pluginChain: ITelemetryPluginChain, config: IConfiguration, core:IAppInsightsCore, startAt?:IPlugin) {
+    constructor(pluginChain: ITelemetryPluginChain, config: IConfiguration, core: IAppInsightsCore, startAt?:IPlugin) {
         let _self = this;
 
         let context = createProcessTelemetryContext(pluginChain, config, core, startAt);

package/src/JavaScriptSDK/TelemetryHelpers.ts

@@ -16,7 +16,7 @@ const strDoUnload = "_doUnload";
 export interface IPluginState {
     core?: IAppInsightsCore;
     isInitialized?: boolean;
-    tearDown?: boolean;
+    teardown?: boolean;
     disabled?: boolean;
 }
 

package/src/JavaScriptSDK.Interfaces/IConfiguration.ts

@@ -84,9 +84,9 @@ export interface IConfiguration {
      * If channels are provided here, core will ignore any channels that are already setup, example if there is a SKU with an initialized channel
      */
     channels?: IChannelControls[][];
+
     /**
      * @type {boolean}
-     * @memberof IConfiguration
      * Flag that disables the Instrumentation Key validation.
      */
     disableInstrumentationKeyValidation?: boolean;
@@ -123,7 +123,6 @@ export interface IConfiguration {
      * @description Custom cookie domain. This is helpful if you want to share Application Insights cookies across subdomains. It
      * can be set here or as part of the cookieCfg.domain, the cookieCfg takes precedence if both are specified.
      * @type {string}
-     * @memberof IConfig
      * @defaultValue ""
      */
     cookieDomain?: string;
@@ -132,7 +131,6 @@ export interface IConfiguration {
      * @description Custom cookie path. This is helpful if you want to share Application Insights cookies behind an application
      * gateway. It can be set here or as part of the cookieCfg.domain, the cookieCfg takes precedence if both are specified.
      * @type {string}
-     * @memberof IConfig
      * @defaultValue ""
      */
     cookiePath?: string;

package/types/JavaScriptSDK/ProcessTelemetryContext.d.ts

@@ -20,7 +20,7 @@ export declare function createProcessTelemetryContext(telemetryChain: ITelemetry
  * @param core - The current core instance
  * @param startAt - Identifies the next plugin to execute, if null there is no "next" plugin and if undefined it should assume the start of the chain
  */
-export declare function createProcessTelemetryUnloadContext(telemetryChain: ITelemetryPluginChain, config: IConfiguration, core: IAppInsightsCore, startAt?: IPlugin): IProcessTelemetryUnloadContext;
+export declare function createProcessTelemetryUnloadContext(telemetryChain: ITelemetryPluginChain, core: IAppInsightsCore, startAt?: IPlugin): IProcessTelemetryUnloadContext;
 /**
  * Creates a new Telemetry Item context with the current config, core and plugin execution chain for updating the configuration
  * @param plugins - The plugin instances that will be executed
@@ -28,7 +28,7 @@ export declare function createProcessTelemetryUnloadContext(telemetryChain: ITel
  * @param core - The current core instance
  * @param startAt - Identifies the next plugin to execute, if null there is no "next" plugin and if undefined it should assume the start of the chain
  */
-export declare function createProcessTelemetryUpdateContext(telemetryChain: ITelemetryPluginChain, config: IConfiguration, core: IAppInsightsCore, startAt?: IPlugin): IProcessTelemetryUpdateContext;
+export declare function createProcessTelemetryUpdateContext(telemetryChain: ITelemetryPluginChain, core: IAppInsightsCore, startAt?: IPlugin): IProcessTelemetryUpdateContext;
 /**
  * Creates an execution chain from the array of plugins
  * @param plugins - The array of plugins that will be executed in this order

package/types/JavaScriptSDK/TelemetryHelpers.d.ts

@@ -6,7 +6,7 @@ import { ITelemetryUnloadState } from "../JavaScriptSDK.Interfaces/ITelemetryUnl
 export interface IPluginState {
     core?: IAppInsightsCore;
     isInitialized?: boolean;
-    tearDown?: boolean;
+    teardown?: boolean;
     disabled?: boolean;
 }
 export declare function _getPluginState(plugin: IPlugin): IPluginState;

package/types/JavaScriptSDK.Interfaces/IConfiguration.d.ts

@@ -73,7 +73,6 @@ export interface IConfiguration {
     channels?: IChannelControls[][];
     /**
      * @type {boolean}
-     * @memberof IConfiguration
      * Flag that disables the Instrumentation Key validation.
      */
     disableInstrumentationKeyValidation?: boolean;
@@ -105,7 +104,6 @@ export interface IConfiguration {
      * @description Custom cookie domain. This is helpful if you want to share Application Insights cookies across subdomains. It
      * can be set here or as part of the cookieCfg.domain, the cookieCfg takes precedence if both are specified.
      * @type {string}
-     * @memberof IConfig
      * @defaultValue ""
      */
     cookieDomain?: string;
@@ -113,7 +111,6 @@ export interface IConfiguration {
      * @description Custom cookie path. This is helpful if you want to share Application Insights cookies behind an application
      * gateway. It can be set here or as part of the cookieCfg.domain, the cookieCfg takes precedence if both are specified.
      * @type {string}
-     * @memberof IConfig
      * @defaultValue ""
      */
     cookiePath?: string;

package/types/tsdoc-metadata.json

@@ -5,7 +5,7 @@
   "toolPackages": [
     {
       "packageName": "@microsoft/api-extractor",
-      "packageVersion": "7.20.0"
+      "packageVersion": "7.22.2"
     }
   ]
 }