Skip to content
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

Servlet 4.0 and HTTP/2 support #757

Open
takezoe opened this issue Nov 14, 2017 · 12 comments
Open

Servlet 4.0 and HTTP/2 support #757

takezoe opened this issue Nov 14, 2017 · 12 comments

Comments

@takezoe
Copy link
Member

takezoe commented Nov 14, 2017

No description provided.

@takezoe takezoe added this to the 2.7.0 milestone Nov 14, 2017
@takezoe takezoe mentioned this issue Nov 23, 2017
24 tasks
@magnolia-k
Copy link
Contributor

magnolia-k commented Jan 8, 2018

tomcat 9 already supports servlet 4.0, but jetty supports version 10.0 for servlet 4.0.
If jetty 10 is not released, servlet 4.0 correspondence will be difficult.

@takezoe
Copy link
Member Author

takezoe commented Jan 11, 2018

@magnolia-k Thanks for advice! It should be postponed. I remove this issue from 2.7.0 milestone.

@takezoe takezoe removed this from the 2.7.0 milestone Jan 11, 2018
@magnolia-k
Copy link
Contributor

https://github.com/eclipse/jetty.project/releases/tag/jetty-10.0.0

Jetty 10.0 has finally been released!

However, Jetty 10 supports Java 11 and above, so if you want to use Jetty 10 based code, you will need Java 11 or above for Scalatra itself.

@takezoe
Copy link
Member Author

takezoe commented Dec 12, 2020

Hmm... I see. Scalatra 2.8 may be a chance to get rid of Java 8 support.

@magnolia-k
Copy link
Contributor

I agree with dropping Java8 support from the new version of Scalatra.

@magnolia-k
Copy link
Contributor

@takezoe

In Jetty 10.0, the WebSocket related code has been refactored and the configuration has changed significantly.
Atmosphere does not yet support Jetty 10, so simply upgrading Jetty will cause the Atmosphere module tests to fail.
(Jetty no longer uses the org.eclipse.jetty.websocket namespace).

This means that in order to upgrade Scalatra's Jetty to 10.0, we will have to wait for Atmosphere to support Jetty 10, but at the current development speed of Atmosphere, it does not seem very likely that this will happen.

This means that we should drop support for Atmosphere in order to make Scalatra Jetty 10.0 compatible.

Also, since Jetty 11.0 changes the javax namespace to jakarta, the basic servlet api will need to be changed, and the metrics module will need to be reviewed.

So, for the time being, upgrading to Jetty 10 seems to be the way to go. s

Translated with www.DeepL.com/Translator (free version)

@takezoe
Copy link
Member Author

takezoe commented Dec 31, 2020

I agree with dropping Atmosphere support. So, our next actions will be:

  • Deprecate atmosphere in Scalatra 2.8
  • Drop Atmosphere support and upgrade to Jetty 10 in Scalatra 2.9

It seems to be a very long way. 😅

@magnolia-k
Copy link
Contributor

Yes!
This is going to take a long time.

Now that we've created a branch for Scalatra 2.8, we'll deprecate it against Atmosphere.
We'll also add an annotation on the Scalatra-Website saying that it will be deprecated in a future version.

@joakime
Copy link

joakime commented Jun 22, 2022

Announced: Community End of Life for Jetty 9.4.x

This is the first step, a "heads up", the next step will be a formal EOL (End of Life), likely in 6 months time.

@DavidPerezIngeniero
Copy link

I'm happy to check that this issue is being worked: 3977647
When it is due approx. a new version of Scalatra updating Jetty and Servlet support?

@joakime
Copy link

joakime commented Feb 26, 2024

Announced: End of Community Support for Jetty 10.0.x and 11.0.x

You should be using Jetty 12 at this point in time.
Note that Jetty 12 still supports the javax.servlet namspace via the ee8 environment option in Jetty 12.

@takezoe
Copy link
Member Author

takezoe commented Feb 26, 2024

Jetty 12 support has been merged in #1632

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants