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

Console output may be lost when using Log4j2 with something that replaces System.out #44380

Closed
wants to merge 1 commit into from

Conversation

nosan
Copy link
Contributor

@nosan nosan commented Feb 20, 2025

See gh-43963

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Feb 20, 2025
@wilkinsona wilkinsona changed the title Register Log42J StatusListener Console output may be lost when using Log4j2 with something that replaces System.out Feb 20, 2025
@wilkinsona wilkinsona added this to the 3.4.3 milestone Feb 20, 2025
@wilkinsona wilkinsona added type: regression A regression from a previous release and removed status: waiting-for-triage An issue we've not yet triaged labels Feb 20, 2025
Update Log42JLoggingSystem to register Boot's StatusListener
instead of resetting the fallbackListener during initialization.

This fixes an issue when resetting the fallback listener leads to
closing standard streams.

See spring-projectsgh-43963

Signed-off-by: Dmytro Nosan <dimanosan@gmail.com>
wilkinsona pushed a commit that referenced this pull request Feb 20, 2025
Update Log42JLoggingSystem to register Boot's StatusListener
instead of resetting the fallbackListener during initialization.

This fixes an issue when resetting the fallback listener leads to
closing standard streams.

See gh-44380

Signed-off-by: Dmytro Nosan <dimanosan@gmail.com>
@nosan
Copy link
Contributor Author

nosan commented Feb 20, 2025

Thanks, @wilkinsona

I did typo in commit message 🤦 (I used Log42j instead of Log4j2)

@wilkinsona
Copy link
Member

Sorry, I would have corrected that if I'd noticed it. It's no big deal though, we've managed plenty of those over the years so welcome to the club.

@Yobyn
Copy link

Yobyn commented Feb 21, 2025

Any idea when 3.4.3 will be available on maven?

@spencergibb
Copy link
Member

It is there already https://repo1.maven.org/maven2/org/springframework/boot/spring-boot-dependencies/3.4.3/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: regression A regression from a previous release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants