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
Fixes #5378 Setting Holders during STARTING #5397
Conversation
Holders are now started/initialized if needed by a new utility method
I thought this issue was unique to Jetty 10? |
@joakime The underlying problem (not being able to add a holder during STARTING) is in 9.4. It only manifested in 10.beta2 because WS changed to add a filter during STARTING. |
The specific change is from PR #5115, where we now lazily create the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In general I'm nervous of this change, but if you add some more tests as per my comments that might help assuage the nervousness.
jetty-servlet/src/test/java/org/eclipse/jetty/servlet/ServletContextHandlerTest.java
Outdated
Show resolved
Hide resolved
jetty-servlet/src/test/java/org/eclipse/jetty/servlet/ServletContextHandlerTest.java
Outdated
Show resolved
Hide resolved
jetty-servlet/src/main/java/org/eclipse/jetty/servlet/ServletHandler.java
Show resolved
Hide resolved
jetty-servlet/src/test/java/org/eclipse/jetty/servlet/ServletContextHandlerTest.java
Outdated
Show resolved
Hide resolved
Holders are now started/initialized if needed by a new utility method
Fixes #5378 Holders are now started/initialized if needed by a new utility method