-
Notifications
You must be signed in to change notification settings - Fork 12
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
Update to Jetty 12 #105
Comments
Maybe it would be worth merging all the current pull requests and make a new breaking release? From my perspective the all benefits motivates the breaking changes. |
Jetty 12 actually supports multiple environments simultaneously:
so we actually wouldn't necessarily have to adopt the new package names. Do folks think we should stick with EE8 or target one of EE9 or EE10? |
@wetneb expressed an opinion on the linked PR that we should go for Jetty 12 with EE8 (Servlet 4.0). Any other opinions? Although the Velocity 2 change is nominally major, there are a bunch of compatibility flags, all of which we've enabled, so it may not actually have an impact on extensions. The other nominally breaking changes are also likely to have low/no impact. |
Jetty 11 implements the Jakarta Servlet 5 API which uses different package naming, requiring changes to Butterfly to support.
It would make sense to bundle some other breaking changes in with this such as an upgrade to Velocity 2.
The text was updated successfully, but these errors were encountered: