-
Notifications
You must be signed in to change notification settings - Fork 72
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.
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
Log messages not showing up in App Insights #142
Comments
In my experience it seemed that Serilog needs time to flush before the azure function stops, maybe try adding a delay? |
@i3anaan it's not an Azure Function so there is plenty of time to flush. |
@johnknoop Even if it is not an azure function, I'd say it is worth to try to add a delay to see if flushing is part of the problem |
This is a long lived process. There's no reason why flushing would be a problem here. Besides, it writes successfully to the other sinks. |
Hmm, I just noticed your
|
FWIW, I had the .WriteTo.ApplicationInsights(TelemetryConfiguration.CreateDefault(), TelemetryConverter.Events) (And am curently trying to move this config to my App Service's Configuration, so far without success.) |
Hi! 👋 Thanks for getting in touch, and sorry we were unable to respond to your request. We're in the process of bringing new maintainers onto this project, and need to close stale tickets so that we can identify which items are still important to users of the sink. If this issue still affects you:
Thanks again! |
Hi
Sorry for the unspecificity of this question, but I've run out of ideas to try.
I've configured this sink using the latest knowledge I could piece together from the readme and various issues in this repo. My application is an AspNetCore 3.0 web app and the host is configured like this:
And my
ConfigureServices
method contains this:I can't figure out why I'm not seeing any traces in App Insights. As you can see, I'm also sending log messages to Logentries, and they turn up as they should.
Anyone who can spot something wrong in my configuration?
Big thanks
The text was updated successfully, but these errors were encountered: