Skip to content
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

Register Logback OnErrorConsoleStatusListener when using custom Logback file #43931

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nosan
Copy link
Contributor

@nosan nosan commented Jan 22, 2025

Prior to this commit, OnErrorConsoleStatusListener was not registered for a custom Logback configuration file.

This commit registers OnErrorConsoleStatusListener when the Logback configuration is loaded from a custom Logback file that does not include any StatusListener.

See gh-43822

It has also been tested with a native image.

A different approach is to register OnErrorConsoleStatusListener right after reportConfigurationErrorsIfNecessary.
main...nosan:spring-boot:43822-1

…ck file

Prior to this commit, OnErrorConsoleStatusListener was not registered
for a custom Logback configuration file.

This commit registers OnErrorConsoleStatusListener when the Logback
configuration is loaded from a custom Logback file that does
not include any StatusListener.

See spring-projectsgh-43822

Signed-off-by: Dmytro Nosan <[email protected]>
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jan 22, 2025
@nosan
Copy link
Contributor Author

nosan commented Jan 24, 2025

Maybe Spring Boot should consistently register OnErrorConsoleStatusListener, regardless of whether the custom Logback configuration defines any StatusListener(s).

main...nosan:spring-boot:43822-2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: waiting-for-triage An issue we've not yet triaged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants