Update telemetry data to add moniker of the detected properties

[vidai-msft]

This pull request includes several changes aimed at improving the handling and representation of detected properties in the sanitizer telemetry. The most important changes include the introduction of a new DetectedPropertiesInfo class, updates to the SanitizerTelemetry class to use this new class, and modifications to methods that interact with detected properties.

Improvements to Detected Properties Handling:

• New DetectedPropertiesInfo Class:

  • Introduced DetectedPropertiesInfo to encapsulate detected properties and provide methods for adding and querying properties. (src/Common/Sanitizer/SanitizerTelemetry.cs, src/Common/Sanitizer/SanitizerTelemetry.csR16-R91)

• Updates to SanitizerTelemetry Class:

  • Modified SanitizerTelemetry to use DetectedPropertiesInfo instead of a HashSet<string>. Added methods to combine detected properties from multiple telemetry instances. (src/Common/Sanitizer/SanitizerTelemetry.cs, src/Common/Sanitizer/SanitizerTelemetry.csL45-R106)
  • Updated the constructor of SanitizerTelemetry to initialize DetectedProperties as an instance of DetectedPropertiesInfo. (src/Common/Sanitizer/SanitizerTelemetry.cs, src/Common/Sanitizer/SanitizerTelemetry.csR16-R91)

Method Modifications:

• WriteSecretsWarningMessage Method:

  • Updated the condition to check if detected properties are empty using the new IsEmpty property of DetectedPropertiesInfo. Adjusted the warning message to use PropertyNames from DetectedPropertiesInfo. (src/Common/AzurePSCmdlet.cs, src/Common/AzurePSCmdlet.csL436-R442)

• PopulateSanitizerPropertiesFromQos Method:

  • Refactored to build a detailed string representation of detected properties using the new DetectedPropertiesInfo class. (src/Common/MetricHelper.cs, src/Common/MetricHelper.csL484-R505)

[vidai-msft]

/azp run azure-powershell-common - ci

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[isra-fel]

LGTM!