-
Notifications
You must be signed in to change notification settings - Fork 407
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
Jetty 9.x will be soon no more maintained #1662
Comments
Another alternative is to use the HTTP server in the JDK: |
@jvermillard thx for sharing this ! I think if we can remove a useless dependencies this is a good thing. |
never tried: https://stackoverflow.com/questions/67915286/java-sse-server-with-com-sun-net-httpserver-httpserver |
I take a look at Of course we could implement our own Too bad, I liked this idea to use JDK directly 😞 |
As I didn't find any good solution... I currently consider to migrate to jetty 12 and so have different minimum java constraint. (not sure how this will affect the build, theoretically not so much as we didn't have any tests based on jetty) This is not ideal but I'm afraid that it will be hard to keep to java8 for all leshan module anyway because more and more libraries move to java 11 or 17... I mean I will not surprised if in the future java minimal version is not the same for each transport layer implementation... 😞 I try to fight against the idea at the back of my mind, that maybe using HttpServer from JDK and write support of SSE and static resource on my own is not so much of code. 😅 |
A try to move to jetty 12 : #1667 |
#1667 is now integrated in So far, so good. Note that we have a documentation about that : https://github.com/eclipse-leshan/leshan/blob/master/documentation/Requirement.md |
source :
UriCompliance.Violation.USER_INFO
to deprecate user info inHttpURI
jetty/jetty.project#12012 (comment)The only supported one will be v12 which use 17 as minimum version ...
So either :
( When we will remove jetty 9.x dependencies, we should remove corresponding CVE in
.trivyignore
)The text was updated successfully, but these errors were encountered: