-
Notifications
You must be signed in to change notification settings - Fork 152
FELIX-6750 Try out Jetty 12.1.0 in Felix HTTP #376
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
base: master
Are you sure you want to change the base?
Conversation
- 1.1.0-SNAPSHOT - Use jetty 12.1.0-alpha1 - Add `org.eclipse.jetty.compression.*` to exports - Add dependency on jetty-compression-gzip (not sure if this is required, maybe only for the client) and jetty-compression-common
- Fix samples
- Move to EE11 - Skip baselining to prevent errors in the baseline phase because of changed API's in 6.1.0
- Pass local maven repo to be able to get to the snapshot version of the servlet-api bundle
- Use snapshot version
- Build servlet-api, as it's used in the Jetty bundles
|
@cziegeler i did a first attempt on getting the http.jetty12 bundle to using Jetty 12.1.0 (servlet API 6.1). Once it's GA we can revisit and check the open ends (like the |
- Update README.md
- Update wrappers to implement new methods for servlet-api 6.1. This will break backwards compatbility though
|
Thanks, thats great. |
# Conflicts: # http/jetty12/src/test/java/org/apache/felix/http/jetty/it/JettySpecificWebsocketIT.java # http/samples/whiteboard/src/main/java/org/apache/felix/http/samples/whiteboard/FelixJettyWebSocketServlet.java # http/samples/whiteboard/src/main/java/org/apache/felix/http/samples/whiteboard/TestWebSocketServletAlternative.java
# Conflicts: # http/itest/pom.xml # http/jetty/pom.xml # http/jetty12/pom.xml # http/samples/whiteboard/pom.xml
- Fix samples
…#375) Bumps [ch.qos.logback:logback-core](https://github.com/qos-ch/logback) from 1.3.12 to 1.3.15. - [Commits](qos-ch/logback@v_1.3.12...v_1.3.15) --- updated-dependencies: - dependency-name: ch.qos.logback:logback-core dependency-type: direct:development ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [ch.qos.logback:logback-core](https://github.com/qos-ch/logback) from 1.3.12 to 1.3.15. - [Release notes](https://github.com/qos-ch/logback/releases) - [Commits](qos-ch/logback@v_1.3.12...v_1.3.15) --- updated-dependencies: - dependency-name: ch.qos.logback:logback-core dependency-type: direct:development ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps org.apache.felix:org.apache.felix.webconsole from 4.0.0 to 4.9.10. --- updated-dependencies: - dependency-name: org.apache.felix:org.apache.felix.webconsole dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
- revert unneeded changes now 6.1.0 already contains the 6.1 changes of the servlet api
- revert unneeded changes now 6.1.0 already contains the 6.1 changes of the servlet api
- revert unneeded changes now 6.1.0 already contains the 6.1 changes of the servlet api
|
Updated to |
- Fix version
- Fix merge - Move to EE11 for ErrorHandler - Use try with resources
- Move to EE11 for new tests - Simplify HTTP client migration
- Organize imports
- Call start - Handle java.nio.channels.AsynchronousCloseException with a FIXME, might be a Jetty bug?
- Fix FIXME, handle async with future
- Reset FIXME, AsynchronousCloseException still happens
# Conflicts: # http/base/pom.xml # http/itest/pom.xml # http/jetty12/pom.xml # http/jetty12/src/test/java/org/apache/felix/http/jetty/it/AbstractJettyTestSupport.java # http/samples/whiteboard/pom.xml
|
Moved to 12.1.0.beta1 |
# Conflicts: # http/base/pom.xml # http/jetty12/pom.xml # http/jetty12/src/test/java/org/apache/felix/http/jetty/it/AbstractJettyTestSupport.java # http/samples/whiteboard/pom.xml Update to 12.1.0.beta2
- Remove jetty-ee
# Conflicts: # http/base/pom.xml # http/jetty12/pom.xml # http/jetty12/src/test/java/org/apache/felix/http/jetty/it/AbstractJettyTestSupport.java # http/samples/whiteboard/pom.xml
Again when switching to 12.1.0 (only on java 21 and 23). |
…s in 12.1.0 (4f69f68) with java 21 and 23
|
@cziegeler now 12.1.0 is out, #376 (comment) might become something to look into. It has been intermittent between beta versions. In the final version, it's again an issue. |
|
maybe we just wait for 12.1.1 :) |
- Use 1.2.0-SNAPSHOT now 1.1.0 will be released soon on master
- Move to 12.1.1
- Fix itest dep
# Conflicts: # http/base/pom.xml # http/jetty12/pom.xml # http/jetty12/src/test/java/org/apache/felix/http/jetty/it/AbstractJettyTestSupport.java # http/samples/whiteboard/pom.xml
# Conflicts: # http/jetty12/pom.xml
- Move to 12.1.3
- Use 1.2.0 snapshot
org.eclipse.jetty.compression.*to exportsThis branch is based on EE11 (servlet API 6.1).