-
Notifications
You must be signed in to change notification settings - Fork 75
Grizzly is not compatible with Servlet 6 #2153
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
Thanks for the reminder, that indeed still needs to be done. That aside, I wonder btw if the half-finished Servlet implementation in Grizzly is really needed still. Is anyone really using this? |
Jersey is using it for the test framework. I think the Jersey test framework is also used by one of the spring boot modules. |
Interesting. Did you ever try running the Servlet TCK against Grizzly to get an idea what percentage of the Servlet API is implemented? And just curiously, but was it really the idea to turn Grizzly into a fully compliant Servlet implementation? |
As far as I remember, Alexey implemented just the minimum functionality for mocking the servlet deployment. It was never meant to be a full servlet implementation. |
metro needs grizzly as well in order to be fully buildable from sources, see eclipse-ee4j/metro-wsit#155 |
So is this issue just requesting a build with the Servlet API brought up to speed with Servlet 6 i.e. not an implementation but something you can compile against? |
from metro point of view - yes; should there be need for anything specific to be implemented, that can be done once we see real failure (ie metro uses grizzly for tcp level transport) |
This issue is about replacing the Servlet 5 deprecated methods that have been removed in Servlet 6 with methods that are not deprecated. |
Just FYI, I am working on that, I hope I will be able to push (first?) changes today. |
#2153 Started development of Grizzly 4
Signed-off-by: David Matějček <[email protected]>
Implemented #2153 - Upgraded to Servlet 6
This issue can be closed now |
jakarta.servlet.SingleThreadModel is no longer in Servlet 6 API.
For instance, WebappContext is not compatible with Servlet 6.
The text was updated successfully, but these errors were encountered: