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

Deadlock Task #102

Open
manuz90flw opened this issue Aug 25, 2023 · 1 comment
Open

Deadlock Task #102

manuz90flw opened this issue Aug 25, 2023 · 1 comment

Comments

@manuz90flw
Copy link

Hi,

I'm using the library on Win Form .NET 4.8 write in vb.net .
The first log arrives to graylog server(5.1.3) but the app stuck.

If I remove GetResult() on src/Serilog.Sinks.Graylog/GraylogSink.cs the app works as i expect.
immagine
@daywee
Copy link
Contributor

daywee commented Sep 8, 2023

Hi, I've encountered the same issue in our asp.net app and using UDP for transport. Normally, it works fine, but when we log very long messages that are chunked, the task gets deadlocked. It seems that there is the same issue as was previously fixed for TCP under #64. I've propagated .ConfigureAwait(false) into underlaying calls to fix the issue under new PR.

Please take a look at #104.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@daywee @manuz90flw