-
Notifications
You must be signed in to change notification settings - Fork 41.2k
Register logging system's shutdown hook by default #25046
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
Comments
I think it makes sense to flip the default now. |
The changes in this branch flip the default. While working on them, it occurred to me that it'd be nice to disable it automatically in a war deployment. We already disable the application's shutdown hook in |
We're going to try setting something via the servlet context in the web application case to disable the shutdown hook. |
The discussion in #24507 makes me think that we should consider enabling the property by default. Whilst the arguments in #4026 are still valid, they aren't really typical Spring Boot deployments.
The text was updated successfully, but these errors were encountered: