azure-container-networking icon indicating copy to clipboard operation
azure-container-networking copied to clipboard

feat: create new telemetry handle that supports connection strings

Open BeegiiK opened this issue 4 months ago • 0 comments

Reason for Change:

Currently, the telemetry handle can only work with instrumentation keys. If we want to use application insights in sovereign clouds, this would simply fail as the logging is shipped to a default ingestion endpoint and the isPublicEnvironment check.

This PR introduces a new telemetry handle method that takes in a connection string, parses it, updates the configuration appropriately and initializes the new handle

Tested manually by updating DNC to use the new telemetry handle and verified that the logs were shipped to Kusto, with the connection string of the test application insights resource:

Issue Fixed:

https://github.com/Azure/azure-container-networking/issues/3730

Requirements:

Notes:

BeegiiK avatar Jun 11 '25 08:06 BeegiiK