Skip to content

MEIER-323: Suppress default ASP.NET Core request logs in favor of Serilog middleware#82

Merged
andymeierdev merged 1 commit intomainfrom
andymeierdev/MEIER-323/suppress-aspnetcore-request-logs
Mar 24, 2026
Merged

MEIER-323: Suppress default ASP.NET Core request logs in favor of Serilog middleware#82
andymeierdev merged 1 commit intomainfrom
andymeierdev/MEIER-323/suppress-aspnetcore-request-logs

Conversation

@andymeierdev
Copy link
Copy Markdown
Collaborator

Summary

Health check logs (Request starting / Request finished for /health) were still appearing in Seq despite the UseSerilogRequestLogging filter added in #81. This is because those logs come from ASP.NET Core's built-in Microsoft.AspNetCore.Hosting.Diagnostics logger, which fires independently of the Serilog request logging middleware.

Fix

Added .MinimumLevel.Override("Microsoft.AspNetCore", LogEventLevel.Warning) to the Serilog configuration. This suppresses all Information-level logs from the Microsoft.AspNetCore namespace (the verbose "Request starting"/"Request finished" pairs), leaving UseSerilogRequestLogging as the sole request logger. Since that middleware already drops /health to Verbose, health checks are now fully silent.

Validation

  • dotnet build — zero warnings/errors
  • All 61 tests pass (./fake.sh Test)

@github-actions
Copy link
Copy Markdown

🍹 preview on andymeier/prod

Pulumi report

View in Pulumi Cloud

  Previewing update (prod)

View Live: https://app.pulumi.com/meiermade/andymeier/prod/previews/7060ba9e-34b2-49f0-99d5-4f39403df2f1

pulumi:pulumi:Stack: (same)
  [urn=urn:pulumi:prod::andymeier::pulumi:pulumi:Stack::andymeier-prod]
  ~ docker-build:index:Image: (update)
      [id=sha256:4220b7870863d07b1de96b565c6d5b26306a51590b5d515cb44734e64b92fa6a]
      [urn=urn:pulumi:prod::andymeier::docker-build:index:Image::andymeier]
    - contextHash: "ffb9077bedc1d0cd2feaa7d73294bd1681c024a1a99666dd717535887cb5b948"
  ~ kubernetes:apps/v1:Deployment: (update)
      [id=andymeier/app]
      [urn=urn:pulumi:prod::andymeier::kubernetes:apps/v1:Deployment::app]
    ~ spec: {
        ~ template: {
            ~ spec: {
                ~ containers: [
                    ~ [0]: {
                            ~ image: "us-east1-docker.pkg.dev/meiermade-platform/platform/andymeier:latest@sha256:77a0d75a144d07f78e41e9bca5cc639e8229bc6cdef35103c5cc72a0cf7b75a4" => [unknown]
                          }
                  ]
              }
          }
      }
Resources:
  ~ 2 to update
  14 unchanged
  

@andymeierdev andymeierdev merged commit 07a499e into main Mar 24, 2026
2 checks passed
@andymeierdev andymeierdev deleted the andymeierdev/MEIER-323/suppress-aspnetcore-request-logs branch March 24, 2026 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant