Skip to content

Commit ddc3091

Browse files
committed
Optimizes daily summary job and email sending
Improves the daily summary job by removing an unnecessary index call, streamlining stack retrieval. Enhances email sending robustness by preventing cancellation exceptions from being caught during health checks, ensuring more accurate error reporting.
1 parent 88ca9d1 commit ddc3091

2 files changed

Lines changed: 2 additions & 3 deletions

File tree

src/Exceptionless.Core/Jobs/DailySummaryJob.cs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -184,8 +184,7 @@ private async Task<bool> SendSummaryNotificationAsync(Project project, SummaryNo
184184
q => q.AppFilter(sf)
185185
.FilterExpression(filter)
186186
.SortExpression("-first")
187-
.DateRange(data.UtcStartTime, data.UtcEndTime, "first")
188-
.Index(data.UtcStartTime, data.UtcEndTime),
187+
.DateRange(data.UtcStartTime, data.UtcEndTime, "first"),
189188
o => o.PageLimit(3));
190189

191190
newest = stackResults.Documents;

src/Exceptionless.Insulation/Mail/MailKitMailSender.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,7 @@ public async Task<HealthCheckResult> CheckHealthAsync(HealthCheckContext context
139139
await client.DisconnectAsync(true, cancellationToken);
140140
_lastSuccessfulConnection = _timeProvider.GetUtcNow().UtcDateTime;
141141
}
142-
catch (Exception ex)
142+
catch (Exception ex) when (ex is not OperationCanceledException)
143143
{
144144
return HealthCheckResult.Unhealthy("Email Not Working.", ex);
145145
}

0 commit comments

Comments
 (0)