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

PAXWEB-1133 - Upgrade to latest Jetty 9.3.x #120

Merged
merged 1 commit into from
Oct 31, 2017
Merged

PAXWEB-1133 - Upgrade to latest Jetty 9.3.x #120

merged 1 commit into from
Oct 31, 2017

Conversation

Maurice-Betzel
Copy link
Contributor

@Maurice-Betzel Maurice-Betzel commented Oct 29, 2017

Enabling Vaadin push on Karaf needed a newer Jetty server. Vaadin push depends on Atmosphere runtime 2.4.11.vaadin2 which needs at least Jetty 9.3.19.v20170502 instead of the 9.3.15.v20161220 that lacks the constructor WebSocketServerFactory(ServletContext, policy) in class org.eclipse.jetty.websocket.server.WebSocketServerFactory.
I upgraded to Jetty 9.3.21.v20170918 and added the missing import-package declaration javax.servlet.annotation; version="[2.3.0,4.0.0)" to pax-web-extender-whiteboard.
This allows annotations like @WebInitParamto to be parsed.

Copy link
Member

@ANierbeck ANierbeck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You may merge yourself if you like

@Maurice-Betzel Maurice-Betzel merged commit 5e9ffe9 into ops4j:master Oct 31, 2017
@kaikreuzer
Copy link

I am also just stuck with the urgent need for a more recent Jetty client, but not finding a way how to get there as openHAB uses Karaf and thus Pax-Web...
I would therefore very much need this update here - is there by chance a new Pax-Web release planned any time soon?

@Maurice-Betzel
Copy link
Contributor Author

You can use my 6.0.7.1 version from https://github.com/Maurice-Betzel/org.ops4j.pax.web-6.0.7.1
It has the changed properties described above.

@ANierbeck
Copy link
Member

I think it's OK to cut a next 6.0.8 release for Pax Web

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

Successfully merging this pull request may close these issues.

3 participants