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
Upgrade to Jetty 9.3.22 #567
Comments
@lolodomo FYI, this is required for the Sonos fix. |
@kaikreuzer : just for a clear understanding and a correct communication, that means the Sonos problem is not yet fixed (even if the issue in ESH is now closed) and still present in the last OH distribution ? |
As summarized here, the fix is to update the Jetty HTTP library and this is what this issue here is about. |
I understand that we recently moved to Jetty 9.3.14 and JUPnP 2.3 which introduced a problem in the Sonos binding due to a bug in Jetty 9.3.14. |
@lolodomo I didn't see any other bindings being hit by the Jetty bug. It seemed to only occur in the massive parallel operation that is done when querying remote UPnP devices for all their services. |
Updating to Jetty 9.3.15 isn't a good choice when using Pax-Web (as done in Karaf), see https://issues.apache.org/jira/plugins/servlet/mobile#issue/KARAF-4990. I therefore decided to go for the latest patch level 9.3.22 instead, which seems to work well. |
Due to eclipse-archived/smarthome#4569 (which fixes eclipse-archived/smarthome#4531), we need to immediately update the Jetty HTTP client to at least version 9.3.15.
Unfortunately, the latest pax-web 6.0.7 release still uses Jetty 9.3.14, so there is no simple way to get a more recent version.
Updating Karaf might help in a way that it defines a "jetty" feature which comes with version 9.3.15 - but this would mean that we have to drop the usage of Pax-Web. I cannot yet fully assess how we can get there and what it exactly means for existing setups.
The text was updated successfully, but these errors were encountered: