Repository navigation
Isolate authentication for multi-endpoint exports - #49875
Radhika Gupta (rads-1996) wants to merge 2 commits into
Conversation
|
The |
|
Azure Pipelines: Successfully started running 1 pipeline(s). 10 pipeline(s) were filtered out due to trigger conditions. There may be pipelines that require an authorized user to comment /azp run to run. |
There was a problem hiding this comment.
🔵 Needs a closer look
Changes to token-audience selection and credential-bearing redirect trust boundaries require human security review under repository guidance before approval.
No open issues 🎉
What changed in this PR
This PR isolates Microsoft Entra authentication for multi-endpoint exports in the Azure Monitor OpenTelemetry exporter while preserving single-endpoint behavior.
Changes:
- Adds routing-specific audience configuration and rejects custom authentication policies in routing mode.
- Restricts routed redirects to trusted destinations and preserves accepted redirect paths.
- Documents shared credential risks and adds regression coverage for authentication and redirects.
| File | Description |
|---|---|
sdk/monitor/azure-monitor-opentelemetry-exporter/tests/test_base_exporter.py |
Tests custom authentication policy handling with and without routing. |
sdk/monitor/azure-monitor-opentelemetry-exporter/tests/logs/test_logs.py |
Covers audience selection, credential isolation, redirect boundaries, and legacy behavior. |
sdk/monitor/azure-monitor-opentelemetry-exporter/README.md |
Documents routed authentication setup, restrictions, and credential risks. |
sdk/monitor/azure-monitor-opentelemetry-exporter/CHANGELOG.md |
Records authentication isolation and redirect fixes. |
sdk/monitor/azure-monitor-opentelemetry-exporter/azure/monitor/opentelemetry/exporter/export/_base.py |
Integrates routing-specific audiences, policy validation, and trusted redirect handling. |
sdk/monitor/azure-monitor-opentelemetry-exporter/azure/monitor/opentelemetry/exporter/_routing.py |
Adds redirect trust validation and complete redirect URL preservation. |
sdk/monitor/azure-monitor-opentelemetry-exporter/azure/monitor/opentelemetry/exporter/_constants.py |
Defines the routed audience environment variable. |
Review effort: Balanced
e26cc44 to
326f497
Compare
Description
All SDK Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines