Skip to content

Conversation

timothycoleman
Copy link
Contributor

When running as a windows service we log to the windows event log, but on shutdown some part of this mechanism in the framework gets shutdown before we are done logging, resulting in scary errors in the regular log about not being able to log.

This also cuts down on the amount of logs being produced.

@timothycoleman timothycoleman requested a review from a team as a code owner September 3, 2025 12:55
@timothycoleman timothycoleman changed the title Avoid adding windows event logging when running as a windows service [DB-1656] Avoid adding windows event logging when running as a windows service Sep 3, 2025
Copy link

linear bot commented Sep 3, 2025

Copy link
Contributor

github-actions bot commented Sep 3, 2025

Qodana for .NET

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

@timothycoleman timothycoleman merged commit be78275 into master Sep 4, 2025
11 of 12 checks passed
@timothycoleman timothycoleman deleted the timothycoleman/windows-service-disable-eventlog-master branch September 4, 2025 07:55
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

@timothycoleman 👉 Created pull request targeting release/v25.0: #5257

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants