applicationinsights.ts

[PhakornKiong]

Fix #837

• pass defaultClient.context to QuickPulseClient

While tinkering with the debugger, I noticed that the constructor for QuickPulseStateManager will use default context if one is not provided.

ApplicationInsights-node.js/Library/QuickPulseStateManager.ts

Lines 35 to 40 in a327c10

	constructor(config: Config, context?: Context) {
	this.config = config;
	this.context = context || new Context();
	this._sender = new QuickPulseSender(this.config);
	this._isEnabled = false;
	}

and

null was always passed to QuickPulseStateManager, hence the default context will always be used

ApplicationInsights-node.js/applicationinsights.ts

Lines 363 to 369 in a327c10

	if (!liveMetricsClient && enable) {
	// No qps client exists. Create one and prepare it to be enabled at .start()
	liveMetricsClient = new QuickPulseClient(defaultClient.config, null);
	_performanceLiveMetrics = new AutoCollectPerformance(liveMetricsClient as any, 1000, true);
	liveMetricsClient.addCollector(_performanceLiveMetrics);
	defaultClient.quickPulseClient = liveMetricsClient; // Need this so we can forward all manual tracks to live metrics via PerformanceMetricsTelemetryProcessor
	} else if (liveMetricsClient) {

This explains #837 inability to get "Role Name" updated in the portal.

[hectorhdzg]

Thanks for your contribution