@microsoft/applicationinsights-core-js 2.7.0-beta.1 → 2.7.1-nightly.202110-04

package/src/JavaScriptSDK/EnvUtils.ts

@@ -2,15 +2,18 @@
 // Licensed under the MIT License.
 "use strict";
 
-import { 
-    getGlobal, strShimUndefined, strShimObject, strShimPrototype, strShimFunction 
+import {
+    getGlobal, strShimUndefined, strShimObject, strShimPrototype
 } from "@microsoft/applicationinsights-shims";
-import { isString, strContains } from "./HelperFuncs";
+import { isString, isUndefined, strContains } from "./HelperFuncs";
+
+// TypeScript removed this interface so we need to declare the global so we can check for it's existence.
+declare var XDomainRequest: any;
 
 /**
  * This file exists to hold environment utilities that are required to check and
  * validate the current operating environment. Unless otherwise required, please
- * only defined methods (functions) in this class so that users of these 
+ * only use defined methods (functions) in this class so that users of these
  * functions/properties only need to include those that are used within their own modules.
  */
 
@@ -31,10 +34,41 @@ const strTrident = "trident/";
 let _isTrident: boolean = null;
 let _navUserAgentCheck: string = null;
 let _enableMocks = false;
+let _useXDomainRequest: boolean | null = null;
+let _beaconsSupported: boolean | null = null;
+
+
+function _hasProperty(theClass: any, property: string) {
+    let supported = false;
+    if (theClass) {
+        try {
+            supported = property in theClass;
+            if (!supported) {
+                let proto = theClass[strShimPrototype];
+                if (proto) {
+                    supported = property in proto;
+                }
+            }
+        } catch (e) {
+            // Do Nothing
+        }
+
+        if (!supported) {
+            try {
+                let tmp = new theClass();
+                supported = !isUndefined(tmp[property]);
+            } catch (e) {
+                // Do Nothing
+            }
+        }
+    }
+
+    return supported;
+}
 
 /**
  * Enable the lookup of test mock objects if requested
- * @param enabled 
+ * @param enabled
  */
 export function setEnableEnvMocks(enabled: boolean) {
     _enableMocks = enabled;
@@ -60,9 +94,9 @@ export function getGlobalInst<T>(name:string): T {
 }
 
 /**
- * Checks if window object is available, this is required as we support the API running without a 
- * window /document (eg. Node server, electron webworkers) and if we attempt to assign a window 
- * object to a local variable or pass as an argument an "Uncaught ReferenceError: window is not defined" 
+ * Checks if window object is available, this is required as we support the API running without a
+ * window /document (eg. Node server, electron webworkers) and if we attempt to assign a window
+ * object to a local variable or pass as an argument an "Uncaught ReferenceError: window is not defined"
  * exception will be thrown.
  * Defined as a function to support lazy / late binding environments.
  */
@@ -85,9 +119,9 @@ export function getWindow(): Window | null {
 }
 
 /**
- * Checks if document object is available, this is required as we support the API running without a 
- * window /document (eg. Node server, electron webworkers) and if we attempt to assign a document 
- * object to a local variable or pass as an argument an "Uncaught ReferenceError: document is not defined" 
+ * Checks if document object is available, this is required as we support the API running without a
+ * window /document (eg. Node server, electron webworkers) and if we attempt to assign a document
+ * object to a local variable or pass as an argument an "Uncaught ReferenceError: document is not defined"
  * exception will be thrown.
  * Defined as a function to support lazy / late binding environments.
  */
@@ -110,9 +144,9 @@ export function getDocument(): Document | null {
 
 
 /**
- * Checks if navigator object is available, this is required as we support the API running without a 
- * window /document (eg. Node server, electron webworkers) and if we attempt to assign a navigator 
- * object to a local variable or pass as an argument an "Uncaught ReferenceError: navigator is not defined" 
+ * Checks if navigator object is available, this is required as we support the API running without a
+ * window /document (eg. Node server, electron webworkers) and if we attempt to assign a navigator
+ * object to a local variable or pass as an argument an "Uncaught ReferenceError: navigator is not defined"
  * exception will be thrown.
  * Defined as a function to support lazy / late binding environments.
  */
@@ -134,9 +168,9 @@ export function getNavigator(): Navigator | null {
 }
 
 /**
- * Checks if history object is available, this is required as we support the API running without a 
- * window /document (eg. Node server, electron webworkers) and if we attempt to assign a history 
- * object to a local variable or pass as an argument an "Uncaught ReferenceError: history is not defined" 
+ * Checks if history object is available, this is required as we support the API running without a
+ * window /document (eg. Node server, electron webworkers) and if we attempt to assign a history
+ * object to a local variable or pass as an argument an "Uncaught ReferenceError: history is not defined"
  * exception will be thrown.
  * Defined as a function to support lazy / late binding environments.
  */
@@ -198,9 +232,9 @@ export function getPerformance(): Performance | null {
 }
 
 /**
- * Checks if JSON object is available, this is required as we support the API running without a 
- * window /document (eg. Node server, electron webworkers) and if we attempt to assign a history 
- * object to a local variable or pass as an argument an "Uncaught ReferenceError: JSON is not defined" 
+ * Checks if JSON object is available, this is required as we support the API running without a
+ * window /document (eg. Node server, electron webworkers) and if we attempt to assign a history
+ * object to a local variable or pass as an argument an "Uncaught ReferenceError: JSON is not defined"
  * exception will be thrown.
  * Defined as a function to support lazy / late binding environments.
  */
@@ -308,8 +342,68 @@ export function isSafari(userAgentStr ?: string) {
     if (!userAgentStr || !isString(userAgentStr)) {
         let navigator = getNavigator() || ({} as Navigator);
         userAgentStr = navigator ? (navigator.userAgent || "").toLowerCase() : "";
-    }    
+    }
 
     var ua = (userAgentStr || "").toLowerCase();
     return (ua.indexOf('safari') >= 0);
 }
+
+/**
+ * Checks if HTML5 Beacons are supported in the current environment.
+ * @returns True if supported, false otherwise.
+ */
+ export function isBeaconsSupported(): boolean {
+    if (_beaconsSupported === null) {
+        _beaconsSupported = hasNavigator() && Boolean(getNavigator().sendBeacon);
+    }
+
+    return _beaconsSupported;
+}
+
+/**
+ * Checks if the Fetch API is supported in the current environment.
+ * @param withKeepAlive - [Optional] If True, check if fetch is available and it supports the keepalive feature, otherwise only check if fetch is supported
+ * @returns True if supported, otherwise false
+ */
+export function isFetchSupported(withKeepAlive?: boolean): boolean {
+    let isSupported = false;
+    try {
+        const fetchApi = getGlobalInst("fetch");
+        isSupported = !!fetchApi;
+        const request = getGlobalInst("Request");
+        if (isSupported && withKeepAlive && request) {
+            isSupported = _hasProperty(request, "keepalive");
+        }
+    } catch (e) {
+        // Just Swallow any failure during availability checks
+    }
+
+    return isSupported;
+}
+
+export function useXDomainRequest(): boolean | undefined {
+    if (_useXDomainRequest === null) {
+        _useXDomainRequest = (typeof XDomainRequest !== undefined);
+        if (_useXDomainRequest && isXhrSupported()) {
+            _useXDomainRequest = _useXDomainRequest && !_hasProperty(getGlobalInst("XMLHttpRequest"), "withCredentials");
+        }
+    }
+
+    return _useXDomainRequest;
+}
+
+/**
+ * Checks if XMLHttpRequest is supported
+ * @returns True if supported, otherwise false
+ */
+export function isXhrSupported(): boolean {
+    let isSupported = false;
+    try {
+        const xmlHttpRequest = getGlobalInst("XMLHttpRequest");
+        isSupported = !!xmlHttpRequest;
+    } catch (e) {
+        // Just Swallow any failure during availability checks
+    }
+
+    return isSupported;
+}

package/src/JavaScriptSDK/HelperFuncs.ts

@@ -1,6 +1,6 @@
 // Copyright (c) Microsoft Corporation. All rights reserved.
 // Licensed under the MIT License.
-import {  
+import {
     strShimUndefined, strShimObject, strShimFunction, throwTypeError,
     ObjClass, ObjProto, ObjAssign, ObjHasOwnProperty, ObjDefineProperty
 } from "@microsoft/applicationinsights-shims";
@@ -24,11 +24,11 @@ export function objToString(obj: any) {
 
 export function isTypeof(value: any, theType: string): boolean {
     return typeof value === theType;
-};
+}
 
 export function isUndefined(value: any): boolean {
     return value === undefined || typeof value === strShimUndefined;
-};
+}
 
 export function isNotUndefined(value: any): boolean {
     return !isUndefined(value);
@@ -44,17 +44,17 @@ export function isNotNullOrUndefined(value: any): boolean {
 
 export function hasOwnProperty(obj: any, prop: string): boolean {
     return obj && ObjHasOwnProperty.call(obj, prop);
-};
+}
 
 export function isObject(value: any): boolean {
     // Changing to inline for performance
     return typeof value === strShimObject;
-};
+}
 
 export function isFunction(value: any): value is Function {
     // Changing to inline for performance
     return typeof value === strShimFunction;
-};
+}
 
 /**
  * Binds the specified function to an event, so that the function gets called whenever the event fires on the object
@@ -73,7 +73,7 @@ export function attachEvent(obj: any, eventNameWithoutOn: string, handlerRef: an
                 obj[strAddEventHelper](eventNameWithoutOn, handlerRef, useCapture);
                 result = true;
             } else if (!isNullOrUndefined(obj[strAttachEvent])) {
-                // IE before version 9                    
+                // IE before version 9
                 obj[strAttachEvent](strOnPrefix + eventNameWithoutOn, handlerRef);
                 result = true;
             }
@@ -124,7 +124,7 @@ export function normalizeJsName(name: string): string {
 }
 
 /**
- * This is a helper function for the equivalent of arForEach(objKeys(target), callbackFn), this is a 
+ * This is a helper function for the equivalent of arForEach(objKeys(target), callbackFn), this is a
  * performance optimization to avoid the creation of a new array for large objects
  * @param target The target object to find and process the keys
  * @param callbackfn The function to call with the details
@@ -165,7 +165,7 @@ export function strEndsWith(value: string, search: string) {
     }
 
     return false;
-}    
+}
 
 /**
  * The strStartsWith() method determines whether a string starts with the characters of the specified string, returning true or false as appropriate.
@@ -302,12 +302,16 @@ export function toISOString(date: Date) {
  */
 export function arrForEach<T>(arr: T[], callbackfn: (value: T, index?: number, array?: T[]) => void|number, thisArg?: any): void {
     let len = arr.length;
-    for (let idx = 0; idx < len; idx++) {
-        if (idx in arr) {
-            if (callbackfn.call(thisArg || arr, arr[idx], idx, arr) === -1) {
-                break;
+    try {
+        for (let idx = 0; idx < len; idx++) {
+            if (idx in arr) {
+                if (callbackfn.call(thisArg || arr, arr[idx], idx, arr) === -1) {
+                    break;
+                }
             }
         }
+    } catch (e) {
+        // This can happen with some native browser objects, but should not happen for the type we are checking for
     }
 }
 
@@ -322,19 +326,23 @@ export function arrForEach<T>(arr: T[], callbackfn: (value: T, index?: number, a
 export function arrIndexOf<T>(arr: T[], searchElement: T, fromIndex?: number): number {
     let len = arr.length;
     let from = fromIndex || 0;
-    for (let lp = Math.max(from >= 0 ? from : len - Math.abs(from), 0); lp < len; lp++) {
-        if (lp in arr && arr[lp] === searchElement) {
-            return lp;
+    try {
+        for (let lp = Math.max(from >= 0 ? from : len - Math.abs(from), 0); lp < len; lp++) {
+            if (lp in arr && arr[lp] === searchElement) {
+                return lp;
+            }
         }
+    } catch (e) {
+        // This can happen with some native browser objects, but should not happen for the type we are checking for
     }
 
     return -1;
 }
 
 /**
- * Calls a defined callback function on each element of an array, and returns an array that contains the results. This helper exists 
- * to avoid adding a polyfil for older browsers that do not define Array.prototype.xxxx (eg. ES3 only, IE8) just in case any page 
- * checks for presence/absence of the prototype implementation. Note: For consistency this will not use the Array.prototype.xxxx 
+ * Calls a defined callback function on each element of an array, and returns an array that contains the results. This helper exists
+ * to avoid adding a polyfil for older browsers that do not define Array.prototype.xxxx (eg. ES3 only, IE8) just in case any page
+ * checks for presence/absence of the prototype implementation. Note: For consistency this will not use the Array.prototype.xxxx
  * implementation if it exists as this would cause a testing requirement to test with and without the implementations
  * @param callbackfn A function that accepts up to three arguments. The map method calls the callbackfn function one time for each element in the array.
  * @param thisArg An object to which the this keyword can refer in the callbackfn function. If thisArg is omitted, undefined is used as the this value.
@@ -344,10 +352,14 @@ export function arrMap<T, R>(arr: T[], callbackfn: (value: T, index?: number, ar
     let _this = thisArg || arr;
     let results = new Array(len);
 
-    for (let lp = 0; lp < len; lp++) {
-        if (lp in arr) {
-            results[lp] = callbackfn.call(_this, arr[lp], arr);
+    try {
+        for (let lp = 0; lp < len; lp++) {
+            if (lp in arr) {
+                results[lp] = callbackfn.call(_this, arr[lp], arr);
+            }
         }
+    } catch (e) {
+        // This can happen with some native browser objects, but should not happen for the type we are checking for
     }
 
     return results;
@@ -356,7 +368,7 @@ export function arrMap<T, R>(arr: T[], callbackfn: (value: T, index?: number, ar
 /**
  * Calls the specified callback function for all the elements in an array. The return value of the callback function is the accumulated result, and is
  * provided as an argument in the next call to the callback function. This helper exists to avoid adding a polyfil for older browsers that do not define
- * Array.prototype.xxxx (eg. ES3 only, IE8) just in case any page checks for presence/absence of the prototype implementation. Note: For consistency 
+ * Array.prototype.xxxx (eg. ES3 only, IE8) just in case any page checks for presence/absence of the prototype implementation. Note: For consistency
  * this will not use the Array.prototype.xxxx implementation if it exists as this would cause a testing requirement to test with and without the implementations
  * @param callbackfn A function that accepts up to four arguments. The reduce method calls the callbackfn function one time for each element in the array.
  * @param initialValue If initialValue is specified, it is used as the initial value to start the accumulation. The first call to the callbackfn function provides this value as an argument instead of an array value.
@@ -410,7 +422,7 @@ let _objKeysDontEnums = [
 ];
 
 /**
- * Returns the names of the enumerable string properties and methods of an object. This helper exists to avoid adding a polyfil for older browsers 
+ * Returns the names of the enumerable string properties and methods of an object. This helper exists to avoid adding a polyfil for older browsers
  * that do not define Object.keys eg. ES3 only, IE8 just in case any page checks for presence/absence of the prototype implementation.
  * Note: For consistency this will not use the Object.keys implementation if it exists as this would cause a testing requirement to test with and without the implementations
  * @param obj Object that contains the properties and methods. This can be an object that you created or an existing Document Object Model (DOM) object.
@@ -578,14 +590,14 @@ export function throwError(message: string): never {
 }
 
 /**
- * Effectively assigns all enumerable properties (not just own properties) and functions (including inherited prototype) from 
+ * Effectively assigns all enumerable properties (not just own properties) and functions (including inherited prototype) from
  * the source object to the target, it attempts to use proxy getters / setters (if possible) and proxy functions to avoid potential
  * implementation issues by assigning prototype functions as instance ones
- * 
+ *
  * This method is the primary method used to "update" the snippet proxy with the ultimate implementations.
- * 
+ *
  * Special ES3 Notes:
- * Updates (setting) of direct property values on the target or indirectly on the source object WILL NOT WORK PROPERLY, updates to the 
+ * Updates (setting) of direct property values on the target or indirectly on the source object WILL NOT WORK PROPERLY, updates to the
  * properties of "referenced" object will work (target.context.newValue = 10 => will be reflected in the source.context as it's the
  * same object). ES3 Failures: assigning target.myProp = 3 -> Won't change source.myProp = 3, likewise the reverse would also fail.
  * @param target - The target object to be assigned with the source properties and functions
@@ -637,7 +649,7 @@ export function proxyAssign(target: any, source: any, chkSet?: (name: string, is
 /**
  * Simpler helper to create a dynamic class that implements the interface and populates the values with the defaults.
  * Only instance properties (hasOwnProperty) values are copied from the defaults to the new instance
- * @param defaults Simple helper 
+ * @param defaults Simple helper
  */
 export function createClassFromInterface<T>(defaults?: T) {
     return class {

package/src/JavaScriptSDK/InstrumentHooks.ts

@@ -14,7 +14,7 @@ const enum CallbackType {
     Response = 1,
     HookError = 2,
     FunctionError = 3
-};
+}
 
 const cbNames = [
     "req", "rsp", "hkErr", "fnErr"

package/src/JavaScriptSDK/NotificationManager.ts

@@ -151,7 +151,7 @@ export class NotificationManager implements INotificationManager {
     /**
      * [Optional] This event is sent if you have enabled perf events, they are primarily used to track internal performance testing and debugging
      * the event can be displayed via the debug plugin extension.
-     * @param perfEvent 
+     * @param perfEvent
      */
     perfEvent?(perfEvent: IPerfEvent): void;
 }

package/src/JavaScriptSDK/PerfManager.ts

@@ -37,7 +37,7 @@ export class PerfEvent implements IPerfEvent {
     isAsync: boolean;
     
     /**
-     * Identifies the total inclusive time spent for this event, including the time spent for child events, 
+     * Identifies the total inclusive time spent for this event, including the time spent for child events,
      * this will be undefined until the event is completed
      */
     time?: number;
@@ -57,7 +57,7 @@ export class PerfEvent implements IPerfEvent {
 
     setCtx?: (key: string, value: any) => void;
 
-    complete: () => void;    
+    complete: () => void;
 
     constructor(name: string, payloadDetails: () => any, isAsync: boolean) {
         let _self = this;
@@ -104,11 +104,11 @@ export class PerfEvent implements IPerfEvent {
                         _self.isChildEvt = (): boolean => true;
                     }
                     _self[key] = value;
-                } 
+                }
                 else if (key === PerfEvent.ChildrenContextKey) {
                     _self[key] = value;
-                } 
-                else 
+                }
+                else
                 {
                     let ctx = _self[strExecutionContextKey] = _self[strExecutionContextKey] || {};
                     ctx[key] = value;
@@ -180,9 +180,9 @@ export class PerfManager implements IPerfManager  {
     }
 
     /**
-     * Create a new event and start timing, the manager may return null/undefined to indicate that it does not 
+     * Create a new event and start timing, the manager may return null/undefined to indicate that it does not
      * want to monitor this source event.
-     * @param src The source name of the event 
+     * @param src The source name of the event
      * @param payloadDetails - An optional callback function to fetch the payload details for the event.
      * @param isAsync - Is the event occurring from a async event
      */
@@ -225,7 +225,7 @@ const doPerfActiveKey = "CoreUtils.doPerf";
  * @param getSource - The callback to create the source name for the event (if perf monitoring is enabled)
  * @param func - The function to call and measure
  * @param details - A function to return the payload details
- * @param isAsync - Is the event / function being call asynchronously or synchronously 
+ * @param isAsync - Is the event / function being call asynchronously or synchronously
  */
 export function doPerf<T>(mgrSource: IPerfManagerProvider | IPerfManager, getSource: () => string, func: (perfEvt?: IPerfEvent) => T, details?: () => any, isAsync?: boolean) {
     if (mgrSource) {

package/src/JavaScriptSDK/ProcessTelemetryContext.ts

@@ -126,7 +126,7 @@ export class ProcessTelemetryContext implements IProcessTelemetryContext {
     /**
      * Call back for telemetry processing before it it is sent
      * @param env - This is the current event being reported
-     * @param itemCtx - This is the context for the current request, ITelemetryPlugin instances 
+     * @param itemCtx - This is the context for the current request, ITelemetryPlugin instances
      * can optionally use this to access the current core instance or define / pass additional information
      * to later plugins (vs appending items to the telemetry item)
      */

package/src/JavaScriptSDK/RandomHelper.ts

@@ -15,7 +15,7 @@ var _mwcZ = 987654321;
 function _mwcSeed(seedValue: number) {
     if (seedValue < 0) {
         // Make sure we end up with a positive number and not -ve one.
-        seedValue >>>= 0;    
+        seedValue >>>= 0;
     }
 
     _mwcW = (123456789 + seedValue) & MaxUInt32;

package/src/JavaScriptSDK/TelemetryHelpers.ts

@@ -49,9 +49,9 @@ export function initializePlugins(processContext:ProcessTelemetryContext, extens
     // Now initiatilize the plugins
     arrForEach(initPlugins, thePlugin => {
         thePlugin.initialize(
-            processContext.getCfg(), 
-            processContext.core(), 
-            extensions, 
+            processContext.getCfg(),
+            processContext.core(),
+            extensions,
             processContext.getNext());
     });
 }
@@ -69,5 +69,5 @@ export function sortPlugins(plugins:IPlugin[]) {
 
         return result;
     });
-    // sort complete    
+    // sort complete
 }

package/src/JavaScriptSDK/TelemetryPluginChain.ts

@@ -10,6 +10,7 @@ import { _InternalLogMessage } from "./DiagnosticLogger";
 import { doPerf } from "./PerfManager";
 import { LoggingSeverity, _InternalMessageId } from '../JavaScriptSDK.Enums/LoggingEnums';
 import { isFunction } from './HelperFuncs';
+import { dumpObj } from './EnvUtils';
 
 export class TelemetryPluginChain implements ITelemetryPluginChain {
 
@@ -32,7 +33,7 @@ export class TelemetryPluginChain implements ITelemetryPluginChain {
     /**
      * Call back for telemetry processing before it it is sent
      * @param env - This is the current event being reported
-     * @param itemCtx - This is the context for the current request, ITelemetryPlugin instances 
+     * @param itemCtx - This is the context for the current request, ITelemetryPlugin instances
      * can optionally use this to access the current core instance or define / pass additional information
      * to later plugins (vs appending items to the telemetry item)
      */
@@ -77,7 +78,7 @@ export class TelemetryPluginChain implements ITelemetryPluginChain {
                     try {
     
                         // Ensure that we keep the context in sync (for processNext()), just in case a plugin
-                        // doesn't calls processTelemetry() instead of itemContext.processNext() or some 
+                        // doesn't calls processTelemetry() instead of itemContext.processNext() or some
                         // other form of error occurred
                         itemCtx.setNext(_nextProxy);
                         if (_hasSetNext) {
@@ -97,7 +98,7 @@ export class TelemetryPluginChain implements ITelemetryPluginChain {
                             itemCtx.diagLog().throwInternal(
                                 LoggingSeverity.CRITICAL,
                                 _InternalMessageId.PluginException,
-                                "Plugin [" + plugin.identifier + "] failed during processTelemetry - " + error);
+                                "Plugin [" + plugin.identifier + "] failed during processTelemetry - " + dumpObj(error));
                         }
     
                         if (_nextProxy && !hasRun) {

package/src/JavaScriptSDK.Enums/EventsDiscardedReason.ts

@@ -27,5 +27,5 @@ export const EventsDiscardedReason = {
     /**
      * The event queue is full.
      */
-    QueueFull: 5,
+    QueueFull: 5
 };

package/src/JavaScriptSDK.Enums/SendRequestReason.ts

@@ -48,4 +48,4 @@ export const enum SendRequestReason {
      * The Maximum number of events have already been queued
      */
     MaxQueuedEvents = 20
-};
+}

package/src/JavaScriptSDK.Interfaces/IAppInsightsCore.ts

@@ -29,7 +29,7 @@ export interface IAppInsightsCore extends IPerfManagerProvider {
 
     /*
     * Initialization queue. Contains functions to run when appInsights initializes
-    */        
+    */
     initialize(config: IConfiguration, extensions: IPlugin[], logger?: IDiagnosticLogger, notificationManager?: INotificationManager): void;
 
     /*
@@ -74,6 +74,8 @@ export interface IAppInsightsCore extends IPerfManagerProvider {
 
     pollInternalLogs?(eventName?: string): number;
 
+    stopPollingInternalLogs?(): void;
+
     /**
      * Return a new instance of the IProcessTelemetryContext for processing events
      */

package/src/JavaScriptSDK.Interfaces/IChannelControls.ts

@@ -11,7 +11,7 @@ export interface IChannelControls extends ITelemetryPlugin {
 
     /**
      * Pause sending data
-     */        
+     */
     pause(): void;
 
     /**
@@ -27,7 +27,7 @@ export interface IChannelControls extends ITelemetryPlugin {
     /**
      * Flush to send data immediately; channel should default to sending data asynchronously
      * @param async: send data asynchronously when true
-     * @param callBack: if specified, notify caller when send is complete 
+     * @param callBack: if specified, notify caller when send is complete
      */
     flush(async: boolean, callBack?: () => void): void;
 }

package/src/JavaScriptSDK.Interfaces/IConfiguration.ts

@@ -83,7 +83,7 @@ export interface IConfiguration {
      * Channel queues that is setup by caller in desired order.
      * 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[][];  
+    channels?: IChannelControls[][];
     /**
      * @type {boolean}
      * @memberof IConfiguration
@@ -102,8 +102,8 @@ export interface IConfiguration {
     enablePerfMgr?: boolean;
 
     /**
-     * [Optional] Callback function that will be called to create a the IPerfManager instance when required and ```enablePerfMgr``` 
-     * is enabled, this enables you to override the default creation of a PerfManager() without needing to ```setPerfMgr()``` 
+     * [Optional] Callback function that will be called to create a the IPerfManager instance when required and ```enablePerfMgr```
+     * is enabled, this enables you to override the default creation of a PerfManager() without needing to ```setPerfMgr()```
      * after initialization.
      */
     createPerfMgr?: (core: IAppInsightsCore, notificationManager: INotificationManager) => IPerfManager;
@@ -120,7 +120,7 @@ export interface IConfiguration {
     idLength?: number;
 
     /**
-     * @description Custom cookie domain. This is helpful if you want to share Application Insights cookies across subdomains. It 
+     * @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
@@ -129,7 +129,7 @@ export interface IConfiguration {
     cookieDomain?: string;
 
     /**
-     * @description Custom cookie path. This is helpful if you want to share Application Insights cookies behind an application 
+     * @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
@@ -144,7 +144,7 @@ export interface IConfiguration {
     disableCookiesUsage?: boolean;
 
     /**
-     * [Optional] A Cookie Manager configuration which includes hooks to allow interception of the get, set and delete cookie 
+     * [Optional] A Cookie Manager configuration which includes hooks to allow interception of the get, set and delete cookie
      * operations. If this configuration is specified any specified enabled and domain properties will take precedence over the
      * cookieDomain and disableCookiesUsage values.
      */

package/src/JavaScriptSDK.Interfaces/ICookieMgr.ts

@@ -14,7 +14,7 @@ export interface ICookieMgr {
     isEnabled(): boolean;
 
     /**
-     * Set the named cookie with the value and optional domain and optional 
+     * Set the named cookie with the value and optional domain and optional
      * @param name - The name of the cookie
      * @param value - The value of the cookie (Must already be encoded)
      * @param maxAgeSec - [optional] The maximum number of SECONDS that this cookie should survive

package/src/JavaScriptSDK.Interfaces/IInstrumentHooks.ts

@@ -62,9 +62,9 @@ export interface IInstrumentCallDetails {
 
     /**
      * This returns an object that the hook function can use to store hook specific
-     * context, it it not shared with any other hook instances and is unique for the 
+     * context, it it not shared with any other hook instances and is unique for the
      * current call.
-     * A hook implementation can use this to pass / share context between different 
+     * A hook implementation can use this to pass / share context between different
      * hook callbacks eg. request/response requst/hookErrors etc.
      */
     ctx: () => any;