Skip to content

Commit 93a3713

Browse files
authored
Address issues with isFeatureEnabled changes (microsoft#2514)
- More fully disable/remove StatsBeat for release
1 parent 4f7f645 commit 93a3713

26 files changed

Lines changed: 1635 additions & 891 deletions

File tree

.aiAutoMinify.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
"_eInternalMessageId",
1414
"SendRequestReason",
1515
"TransportType",
16+
"eStatsType",
1617
"TelemetryUnloadReason",
1718
"TelemetryUpdateReason"
1819
]

AISKU/src/AISku.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -377,17 +377,17 @@ export class AppInsightsSku implements IApplicationInsights {
377377
_throttleMgr.onReadyState(true);
378378
}
379379

380-
if (!_iKeySentMessage && !_config.connectionString && isFeatureEnabled(IKEY_USAGE, _config)) {
380+
if (!_iKeySentMessage && !_config.connectionString && isFeatureEnabled(IKEY_USAGE, _config, true)) {
381381
_throttleMgr.sendMessage( _eInternalMessageId.InstrumentationKeyDeprecation, "See Instrumentation key support at aka.ms/IkeyMigrate");
382382
_iKeySentMessage = true;
383383
}
384384

385-
if (!_cdnSentMessage && _self.context.internal.sdkSrc && _self.context.internal.sdkSrc.indexOf("az416426") != -1 && isFeatureEnabled(CDN_USAGE, _config)) {
385+
if (!_cdnSentMessage && _self.context.internal.sdkSrc && _self.context.internal.sdkSrc.indexOf("az416426") != -1 && isFeatureEnabled(CDN_USAGE, _config, true)) {
386386
_throttleMgr.sendMessage( _eInternalMessageId.CdnDeprecation, "See Cdn support notice at aka.ms/JsActiveCdn");
387387
_cdnSentMessage = true;
388388
}
389389

390-
if (!_sdkVerSentMessage && parseInt(_snippetVersion) < 6 && isFeatureEnabled(SDK_LOADER_VER, _config)) {
390+
if (!_sdkVerSentMessage && parseInt(_snippetVersion) < 6 && isFeatureEnabled(SDK_LOADER_VER, _config, true)) {
391391
_throttleMgr.sendMessage( _eInternalMessageId.SdkLdrUpdate, "An updated Sdk Loader is available, see aka.ms/SnippetVer");
392392
_sdkVerSentMessage = true;
393393
}

channels/1ds-post-js/src/HttpManager.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -227,7 +227,7 @@ export class HttpManager {
227227
const csStream = getInst("CompressionStream");
228228

229229
// Controls whether payload compression (gzip) is enabled.
230-
_zipPayload = isFeatureEnabled("zipPayload", coreConfig);
230+
_zipPayload = isFeatureEnabled("zipPayload", coreConfig, false);
231231
// if user has payload processor (_sendHook), they may compress the payload themselves
232232
// to avoid double compression, we should disable the zipPayload
233233
if (!isFunction(csStream) || _sendHook) {

0 commit comments

Comments
 (0)