search

Xabaril / AspNetCore.Diagnostics.HealthChecks

Enterprise HealthChecks for ASP.NET Core Diagnostics Package
Apache License 2.0
4.06k stars 793 forks source link

SendGrid Health Check needs to change the log levels #2277

Open DavidThielen opened 2 months ago

DavidThielen commented 2 months ago

Using the SendGrid HealthCheck, I get the following every minute:

Information 00:03:32.822 [wn0xsdwk0000HI]-[] System.Net.Http.HttpClient.sendgrid.LogicalHandler - Start processing HTTP request POST https://api.sendgrid.com/v3/mail/send
Information 00:03:32.822 [wn0xsdwk0000HI]-[] System.Net.Http.HttpClient.sendgrid.ClientHandler - Sending HTTP request POST https://api.sendgrid.com/v3/mail/send
Information 00:03:32.943 [wn0xsdwk0000HI]-[] System.Net.Http.HttpClient.sendgrid.ClientHandler - Received HTTP response headers after 120.9307ms - 200
Information 00:03:32.943 [wn0xsdwk0000HI]-[] System.Net.Http.HttpClient.sendgrid.LogicalHandler - End processing HTTP request after 121.3726ms - 200

These log messages, in my opinion, should be Trace. Having 4 messages every minute to say things are fine should never be an Information level message.