You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It would be nice if this phase was declared as a public constant. This would make it easier to start a custom SmartLifecycle before an application starts accepting HTTP requests.
Currently we declare our own constant for this, but there is no guarantee the WebServerStartStopLifecycle phase will never change.
It is also not possible to depend on the webServerStartStop bean in order to obtain its phase, since dependency relationships take precedence over phases. That is, if Bean A depends on the webServerStartStop bean, it will be started after the webServerStartStop bean, regardless of Bean A's phase.
The text was updated successfully, but these errors were encountered:
WebServerStartStopLifecycle is a package-private class so adding a constant there won't help. Perhaps we could add something to ServletWebServerApplicationContext?
There's the reactive side of things as well so I don't think it should be added to ServletWebServerApplicationContext . Perhaps WebServerApplicationContext though? The constant for the graceful shutdown phase could move there too.
Uh oh!
There was an error while loading. Please reload this page.
The (servlet and reactive)
WebServerStartStopLifecycle.getPhase()
method is currently hardcoded as:It would be nice if this phase was declared as a public constant. This would make it easier to start a custom
SmartLifecycle
before an application starts accepting HTTP requests.Currently we declare our own constant for this, but there is no guarantee the
WebServerStartStopLifecycle
phase will never change.It is also not possible to depend on the
webServerStartStop
bean in order to obtain its phase, since dependency relationships take precedence over phases. That is, if Bean A depends on thewebServerStartStop
bean, it will be started after thewebServerStartStop
bean, regardless of Bean A's phase.The text was updated successfully, but these errors were encountered: