Skip to content

Upgrade jupnp to new version using modularized transport - #5740

Draft
holgerfriedrich wants to merge 3 commits into
openhab:mainfrom
holgerfriedrich:pr-jupnp
Draft

Upgrade jupnp to new version using modularized transport#5740
holgerfriedrich wants to merge 3 commits into
openhab:mainfrom
holgerfriedrich:pr-jupnp

Conversation

@holgerfriedrich

Copy link
Copy Markdown
Member

This upgrades OH to work with the proposed separation of transport layers in jupnp (required for transition to jetty12).

This PR is used for testing and need to be updated once jupnp releases.

@wborn FYI

Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
The Jetty 9 transport falls through to its own standalone StreamServer
when no shared-server provider is registered, opening a second socket
on the UPnP callback port (default 8080) instead of sharing openHAB's
existing HttpService - colliding with pax-web on that port. Add
org.jupnp.transport.httpservice so OSGiUpnpServiceConfiguration uses
the shared HttpService path instead.

Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting external dependency Depends on a new version of an external dependency

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant