Skip to content

Commit 299b12a

Browse files
revert change
1 parent 99c9850 commit 299b12a

File tree

4 files changed

+1
-33
lines changed

4 files changed

+1
-33
lines changed

src/AzureAppConfigurationImpl.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ export class AzureAppConfigurationImpl implements AzureAppConfiguration {
6363
this.#options = options;
6464

6565
// Enable request tracing if not opt-out
66-
this.#requestTracingEnabled = options?.requestTracingOptions?.enabled ?? requestTracingEnabled();
66+
this.#requestTracingEnabled = requestTracingEnabled();
6767

6868
if (options?.trimKeyPrefixes) {
6969
this.#sortedTrimKeyPrefixes = [...options.trimKeyPrefixes].sort((a, b) => b.localeCompare(a));

src/AzureAppConfigurationOptions.ts

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ import { KeyVaultOptions } from "./keyvault/KeyVaultOptions.js";
66
import { RefreshOptions } from "./RefreshOptions.js";
77
import { SettingSelector } from "./types.js";
88
import { FeatureFlagOptions } from "./featureManagement/FeatureFlagOptions.js";
9-
import { RequestTracingOptions } from "./requestTracing/RequestTracingOptions.js";
109

1110
export const MaxRetries = 2;
1211
export const MaxRetryDelayInMs = 60000;
@@ -48,9 +47,4 @@ export interface AzureAppConfigurationOptions {
4847
* Specifies options used to configure feature flags.
4948
*/
5049
featureFlagOptions?: FeatureFlagOptions;
51-
52-
/**
53-
* Specifies options used to configure request tracing.
54-
*/
55-
requestTracingOptions?: RequestTracingOptions;
5650
}

src/requestTracing/RequestTracingOptions.ts

Lines changed: 0 additions & 12 deletions
This file was deleted.

test/requestTracing.test.ts

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -122,20 +122,6 @@ describe("request tracing", function () {
122122
delete process.env.AZURE_APP_CONFIGURATION_TRACING_DISABLED;
123123
});
124124

125-
it("should disable request tracing by RequestTracingOptions", async () => {
126-
try {
127-
await load(createMockedConnectionString(fakeEndpoint), {
128-
clientOptions,
129-
requestTracingOptions: {
130-
enabled: false
131-
}
132-
});
133-
} catch (e) { /* empty */ }
134-
expect(headerPolicy.headers).not.undefined;
135-
const correlationContext = headerPolicy.headers.get("Correlation-Context");
136-
expect(correlationContext).undefined;
137-
});
138-
139125
it("should have request type in correlation-context header when refresh is enabled", async () => {
140126
mockAppConfigurationClientListConfigurationSettings([{
141127
key: "app.settings.fontColor",

0 commit comments

Comments
 (0)