Skip to content

1347 do away with serilog#1348

Merged
aharwood2 merged 3 commits into
devfrom
1347-do-away-with-serilog
Jun 9, 2026
Merged

1347 do away with serilog#1348
aharwood2 merged 3 commits into
devfrom
1347-do-away-with-serilog

Conversation

@aharwood2

@aharwood2 aharwood2 commented May 28, 2026

Copy link
Copy Markdown
Member

This pull request updates the logging configuration for the application, simplifying the setup by removing Serilog and standardizing on console logging. It also adjusts logging levels for certain components in the production settings.

Logging configuration changes:

  • Removed all Serilog-related packages from PPMTool.csproj, including Serilog, Serilog.Sinks.File, Serilog.Sinks.Console, and Serilog.Extensions.Logging.
  • Replaced the Serilog-based logging setup in Program.cs with a simpler configuration that uses only the built-in console logger. All other logging providers are cleared before adding the console logger.

Logging level adjustments:

  • Added a warning-level log filter for Microsoft.EntityFrameworkCore in appsettings.Production.json to reduce log verbosity in production.

@aharwood2 aharwood2 added this to the v1.14.1 milestone May 28, 2026
@aharwood2 aharwood2 self-assigned this May 28, 2026
@aharwood2 aharwood2 linked an issue May 28, 2026 that may be closed by this pull request
@aharwood2 aharwood2 modified the milestones: v1.14.1, v1.14.0.1 Jun 8, 2026

@PhilBradbury PhilBradbury left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good from the code. Tricky to test easily until it's pushed to the Dev server.

@aharwood2 aharwood2 merged commit a60bd01 into dev Jun 9, 2026
12 checks passed
@aharwood2 aharwood2 deleted the 1347-do-away-with-serilog branch June 9, 2026 11:05
@aharwood2 aharwood2 modified the milestones: v1.14.0.1, v1.14.1 Jun 15, 2026
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.

Do away with SeriLog

2 participants