Skip to content

Debug logger should print exceptions even if the formatted message is empty #116339

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jun 6, 2025

Conversation

colejohnson66
Copy link
Contributor

As the title says.

Fixes #116314.

Now prints the exception even if the formatted message is empty.
@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Jun 5, 2025
Copy link
Contributor

Tagging subscribers to this area: @dotnet/area-extensions-logging
See info in area-owners.md if you want to be subscribed.

@tarekgh tarekgh added this to the 10.0.0 milestone Jun 5, 2025
colejohnson66 and others added 2 commits June 5, 2025 13:06
Assert exception is not null in one branch
@tarekgh
Copy link
Member

tarekgh commented Jun 5, 2025

@colejohnson66 please don't merge main branch again, it just reset the CI runs. Thanks!

@colejohnson66
Copy link
Contributor Author

Ah. Didn't know that. Got it 👍🏻

Copy link
Member

@tarekgh tarekgh left a comment

Choose a reason for hiding this comment

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

@tarekgh tarekgh merged commit 6388297 into dotnet:main Jun 6, 2025
84 of 86 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-Extensions-Logging community-contribution Indicates that the PR has been added by a community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Debug logger does not print if the formatted message is empty, even if an exception is provided
2 participants