This repository has been archived by the owner on Nov 1, 2022. It is now read-only.
LogEvent uses a context.TODO()
#775
Labels
context.TODO()
#775
The upstream API method
LogEvent
accepts a context, but when we use it we passcontext..TODO()
. I left it like this because it was not trivial, and not in the critical path of #759.At some point it ought to be changed so that a context is passed through -- the main impediment is that the method also forms part of the EventWriter interface, where it doesn't make as much sense to include a context. It may turn out that interface isn't really necessary though.
The text was updated successfully, but these errors were encountered: