SerilogSinksInMemory icon indicating copy to clipboard operation
SerilogSinksInMemory copied to clipboard

Create a thread-safe InMemorySink

Open gulimran opened this issue 3 years ago • 2 comments

We would like to create a thread-safe InMemorySink for the use in our projects. At the moment, when we make a few parallel async calls, the logger does not have all the log events.

We see two possible solutions:

  1. Add a lock object to make all methods thread-safe in the existing InMemorySink class.
  2. Same as above but in a new thread-safe class which can be accessed by a new builder method in the InMemorySinkExtensions class.

Please let me know which one is a preferred option and we are happy to raise a pull request.

gulimran avatar Jun 23 '21 14:06 gulimran

@gulimran let me have a think about this. It’s been something that’s been on my todo list for a while but I’d like to make sure the usage is going to be predictable for people using the package

sandermvanvliet avatar Jun 23 '21 18:06 sandermvanvliet

I’ll check locally becauseIIRC I had a branch that already had some parts of this

sandermvanvliet avatar Jun 23 '21 18:06 sandermvanvliet