azure-container-networking
azure-container-networking copied to clipboard
feat: create new telemetry handle that supports connection strings
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:
- [x] uses conventional commit messages
- [ ] includes documentation
- [x] adds unit tests
- [ ] relevant PR labels added
Notes: