-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
http2-hpack not visible in OSGi #1290
Comments
Fixed in jetty-9.4.x master ready for 9.4.2 release. |
This causes issues for Eclipse Platform. See https://bugs.eclipse.org/bugs/show_bug.cgi?id=512807 . Can we workaround it on our side somehow to not distribute apache aries bundles too? |
Out of curiosity, how does the Eclipse Help system operate with HTTP/2? Perhaps the Eclipse Help system should not rely on HTTP/2? |
@akurtakov Jetty 9.4.2.v20170220 has been released. |
Eclipse does not use http2, we use the "jetty-http" bundle only. |
@mbooth101 then exclude all of the http2 jars/bundles/components, they are not needed/used, and are optional for Jetty. |
@joakime As discussed in the Eclipse bug quoted by akurtakov above ( https://bugs.eclipse.org/bugs/show_bug.cgi?id=512807 ) the problem is that this change added the following non-optional dep to the jetty-http bundle: Require-Capability: osgi.extender;filter:="osgi.extender=osgi.serviceloader.processor)" It's not a case of excluding http2 bundles, it's a case of this new requirement causing bundle resolution failures because a new non-optional dependency was added to jetty-http. Since Eclipse can use jetty fine without Apache Aries bundles, we believe this dependency should be made optional. Let me submit a PR to illustrate what change I would like to see made. |
Please see PR #1365 |
jetty resolved hpack issue here, so isn't need work-around jetty/jetty.project#1290 Atmosphere don't compactible with new jetty Atmosphere/atmosphere#2242 SessionManager is removed: http://www.eclipse.org/jetty/documentation/9.4.x/session-management.html http://www.eclipse.org/jetty/documentation/9.3.0.v20150612/session-management.html
jetty resolved hpack issue here, so isn't need work-around jetty/jetty.project#1290 Atmosphere don't compactible with new jetty Atmosphere/atmosphere#2242 SessionManager is removed: http://www.eclipse.org/jetty/documentation/9.4.x/session-management.html http://www.eclipse.org/jetty/documentation/9.3.0.v20150612/session-management.html
Reported on stackoverflow: http://stackoverflow.com/questions/41870597/jetty-9-http-2-in-osgi-container-cannot-load-a-httpfieldpreencoder-implementati
The text was updated successfully, but these errors were encountered: