Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[FEATURE REQ] Set client LocationIP field by default when using Azure.Monitor.OpenTelemetry.AspNetCore #44687

Open
vishweshbankwar opened this issue Jun 21, 2024 · 1 comment
Open

[FEATURE REQ] Set client LocationIP field by default when using Azure.Monitor.OpenTelemetry.AspNetCore #44687

vishweshbankwar opened this issue Jun 21, 2024 · 1 comment
Labels
Client This issue points to a problem in the data-plane of the library. Monitor - Distro Monitor OpenTelemetry Distro needs-team-attention Workflow: This issue needs attention from Azure service team or SDK team Service Attention Workflow: This issue is responsible by Azure service team.

Comments

@vishweshbankwar
Copy link
Contributor

Library name

Azure.Monitor.OpenTelemetry.AspNetCore

Please describe the feature.

Related to: #42850

When using Application Insights SDK, the location IP field is set by default. When using Azure.Monitor.OpenTelemetry.AspNetCore, users need to enable this collection by setting client.address attribute on the activity using Enrich or processor. https://learn.microsoft.com/en-us/azure/azure-monitor/app/opentelemetry-add-modify?tabs=aspnetcore#set-the-user-ip.

Opening this issue to enable the collection of LocationIP by default when using Azure.Monitor.OpenTelemetry.AspNetCore
@github-actions github-actions bot added Client This issue points to a problem in the data-plane of the library. Monitor - Exporter Monitor OpenTelemetry Exporter needs-team-attention Workflow: This issue needs attention from Azure service team or SDK team Service Attention Workflow: This issue is responsible by Azure service team. labels Jun 21, 2024
@github-actions GitHub Actions
Copy link

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @cijothomas @rajkumar-rangaraj @reyang @TimothyMothra @vishweshbankwar.

@vishweshbankwar vishweshbankwar mentioned this issue Jun 21, 2024
Closed
@vishweshbankwar vishweshbankwar added Monitor - Distro Monitor OpenTelemetry Distro and removed Monitor - Exporter Monitor OpenTelemetry Exporter labels Jun 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Client This issue points to a problem in the data-plane of the library. Monitor - Distro Monitor OpenTelemetry Distro needs-team-attention Workflow: This issue needs attention from Azure service team or SDK team Service Attention Workflow: This issue is responsible by Azure service team.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@vishweshbankwar