Skip to content

Commit 75718d4

Browse files
committed
simplify activity code
1 parent b9d2ed8 commit 75718d4

File tree

2 files changed

+5
-23
lines changed

2 files changed

+5
-23
lines changed

src/Roastery/Data/Database.cs

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -152,14 +152,8 @@ async Task LogExecAsync(string sql, int rowCount)
152152

153153
var delay = 10 + (int)(Distribution.Uniform() * Math.Pow(rowCount, 1.6));
154154
await Task.Delay(delay);
155-
156-
if (activity?.Activity != null)
157-
{
158-
if (_logger.BindProperty("RowCount", rowCount, false, out var rowCountProperty))
159-
{
160-
ActivityInstrumentation.SetLogEventProperty(activity.Activity, rowCountProperty);
161-
}
162-
}
155+
156+
activity.AddProperty("RowCount", rowCount);
163157
}
164158

165159
static T Clone<T>(T value) where T: IIdentifiable, new()

src/Roastery/Web/RequestLoggingMiddleware.cs

Lines changed: 3 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -43,24 +43,12 @@ public override async Task<HttpResponse> InvokeAsync(HttpRequest request)
4343
}
4444
}
4545

46-
bool LogCompletion(LoggerActivity? activity, Exception? exception, HttpStatusCode statusCode)
46+
bool LogCompletion(LoggerActivity activity, Exception? exception, HttpStatusCode statusCode)
4747
{
4848
var level = (int)statusCode >= 500 ? LogEventLevel.Error : LogEventLevel.Information;
49-
if (activity?.Activity != null)
50-
{
51-
if (_logger.BindProperty("StatusCode", (int)statusCode, false, out var statusCodeProperty))
52-
{
53-
ActivityInstrumentation.SetLogEventProperty(activity.Activity,
54-
statusCodeProperty);
55-
}
56-
57-
if (exception != null)
58-
{
59-
ActivityInstrumentation.TrySetException(activity.Activity, exception);
60-
}
61-
}
6249

63-
activity?.Complete(level);
50+
activity.AddProperty("StatusCode", (int)statusCode);
51+
activity.Complete(level, exception);
6452

6553
return false;
6654
}

0 commit comments

Comments
 (0)