Merge pull request #76 from nblumhardt/fix-75

Consistently use `*Async()` methods so that background level checks occur
datalust · Jul 25, 2017 · c5c54a5 · c5c54a5
2 parents bd714cc + 437de68
commit c5c54a5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/sample/Sample/Program.cs b/sample/Sample/Program.cs
@@ -17,15 +17,15 @@ public static void Main(string[] args)
                 .MinimumLevel.ControlledBy(levelSwitch)
                 .WriteTo.LiterateConsole()
                 .WriteTo.Seq("http://localhost:5341",
-                             apiKey: "yeEZyL3SMcxEKUijBjN",
+                             apiKey: "o6nYf3WWnzF43Uu5PZWJ",
                              controlLevelSwitch: levelSwitch)
                 .CreateLogger();
 
             Log.Information("Sample starting up");
 
             foreach (var i in Enumerable.Range(0, 1000))
             {
-                Log.Information("Running loop {Counter}", i);
+                Log.Information("Running loop {Counter}, switch is at {Level}", i, levelSwitch.MinimumLevel);
 
                 Thread.Sleep(1000);
                 Log.Debug("Loop iteration done");

diff --git a/src/Serilog.Sinks.Seq/Sinks/Seq/SeqSink.cs b/src/Serilog.Sinks.Seq/Sinks/Seq/SeqSink.cs
@@ -76,12 +76,12 @@ protected override void Dispose(bool disposing)
 
         // The sink must emit at least one event on startup, and the server be
         // configured to set a specific level, before background level checks will be performed.
-        protected override void OnEmptyBatch()
+        protected override async Task OnEmptyBatchAsync()
         {
             if (_controlledSwitch.IsActive &&
                 _nextRequiredLevelCheckUtc < DateTime.UtcNow)
             {
-                EmitBatch(Enumerable.Empty<LogEvent>());
+                await EmitBatchAsync(Enumerable.Empty<LogEvent>());
             }
         }