liz/PluralKit
liz/PluralKit
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Yield event handlers to task queue before handling

Browse Source
This commit is contained in:
Ske 2019-08-12 01:15:55 +02:00
parent a689d61788
commit ea476d4051
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
PluralKit.Bot/Bot.cs
View File

@ -254,6 +254,10 @@ namespace PluralKit.Bot
// Inner function so we can await the handler without stalling the entire pipeline // Inner function so we can await the handler without stalling the entire pipeline
async Task Inner() async Task Inner()
{ {
// "Fork" this task off by ~~yeeting~~ yielding it at the back of the task queue
// This prevents any synchronous nonsense from also stalling the pipeline before the first await point
await Task.Yield();
// Create a DI scope for this event // Create a DI scope for this event
// and log the breadcrumb to the newly created (in-svc-scope) Sentry scope // and log the breadcrumb to the newly created (in-svc-scope) Sentry scope
using (var scope = _services.CreateScope()) using (var scope = _services.CreateScope())