Skip to content
This repository has been archived by the owner on Jun 10, 2020. It is now read-only.

Support EventId in ApplicationInsightsLogger #569

Closed
duracellko opened this issue Dec 12, 2017 · 2 comments
Closed

Support EventId in ApplicationInsightsLogger #569

duracellko opened this issue Dec 12, 2017 · 2 comments
Labels
Milestone

Comments

@duracellko
Copy link
Contributor

It would be nice, if ApplicationInsightsLogger would include EventId.Id and EventId.Name in telemetry properties ad EventId and EventName.

@older
Copy link

older commented May 7, 2019

What is official way to enable EventId today? I used documentation from https://github.com/Microsoft/ApplicationInsights-aspnetcore/wiki/Logging, but it is now deprecated and recommended replacement Microsoft.Extensions.Logging.ApplicationInsights.ApplicationInsightsLoggerOptions does not even have IncludeEventId property.

@cijothomas
Copy link
Contributor

@older In the new provider Microsoft.Extensions.Logging.ApplicationInsightsLoggingProvider, EventID and EventName are always included. You dont need to do anything extra to enable it.

Please give it a try, and if EventId/Name are not captured, let us know.
https://docs.microsoft.com/en-us/azure/azure-monitor/app/ilogger

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants