Make URL Redaction more dynamic

[rads-1996]

No description provided.

[Copilot]

Pull request overview

This PR updates the URL field redaction configuration in AppInsightsCore by separating control of credential redaction vs query-parameter redaction, and expands unit test coverage for the new configuration behavior.

Changes:

• Introduces separate query-parameter redaction configuration (redactQueryParams boolean + append/replace query param lists).
• Updates fieldRedaction() / query-parameter redaction selection logic to respect the new configuration shape.
• Adjusts and adds unit tests covering disabled redaction, credential-only redaction, and custom query param modes.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File	Description
shared/AppInsightsCore/Tests/Unit/src/ai/ApplicationInsightsCore.Tests.ts	Adds/updates FieldRedaction unit tests for new query-param config modes and flag interactions.
shared/AppInsightsCore/Tests/Unit/src/ai/AppInsightsCommon.tests.ts	Updates dataSanitizeUrl test config usage and adds coverage for “replace default sensitive params” behavior.
shared/AppInsightsCore/src/utils/EnvUtils.ts	Updates query-param redaction selection and splits credential vs query redaction behavior in fieldRedaction().
shared/AppInsightsCore/src/interfaces/ai/IConfiguration.ts	Changes the public config surface for URL redaction (new boolean + new append/replace arrays).

Comments suppressed due to low confidence (1)

shared/AppInsightsCore/Tests/Unit/src/ai/ApplicationInsightsCore.Tests.ts:2264

• Test name/assert message say this covers an "empty redactQueryParams array", but the test now uses let config = {} as IConfiguration; (no empty array / no query-param config at all). Either pass an explicit empty list (for the new config shape) or rename the test/message so it matches what’s being exercised.

        this.testCase({
            name: "FieldRedaction: should handle empty redactQueryParams array",
            test: () => {
                let config = {} as IConfiguration;
                
                // Should still redact default parameters
                const url = "https://example.com/path?Signature=secret&custom_param=value";
                const redactedLocation = fieldRedaction(url, config);
                Assert.equal(redactedLocation, "https://example.com/path?Signature=REDACTED&custom_param=value",
                    "URL with default sensitive parameters should still be redacted with empty custom array");

---

You can also share your feedback on Copilot code review. Take the survey.

[rads-1996]

Will document all the changes once decide all the config changes.

[MSNev]

these ones should be referencing the eUrlRedactOptions so that during conversion from TypeScript to JavaScript TS will replace with the "number", by referencing the type the code is not minifyable

[MSNev]

Same again use the const enum eUrlRedactOptions

[MSNev]

Add all other references

[rads-1996]

Add all other references

Did you mean "And all other references"

[MSNev]

A couple of minor fixes for minification purposes -- should not affect the functionality