Skip to content
This repository was archived by the owner on May 28, 2018. It is now read-only.

Fixes error where Netty was not binding properly to a host. #261

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fixes error where Netty was not binding properly to a host. #261

wants to merge 1 commit into from

Conversation

ccleve
Copy link

@ccleve ccleve commented Feb 9, 2017

The NettyHttpContainerProvider currently ignores the "host" part of the baseUri. This is ok when you want Netty to bind to the main network interface on a server, but it's a problem when there are multiple interfaces, or you want to do loopback testing. For example, you might want to have one instance of Netty bind to 127.0.0.1 and a second one bind to 127.0.0.2 so you can test communications between the two.

@pavelbucek
Copy link
Member

Thanks for your contribution!

We cannot evaluate and accept your proposal without you signing Oracle Contributor Agreement: http://www.oracle.com/technetwork/community/oca-486395.html. Please let us know whether you plan to do that.

Best regards,
Pavel

@ccleve
Copy link
Author

ccleve commented Feb 23, 2017

I just send the OCA.

@ccleve
Copy link
Author

ccleve commented May 10, 2017

Any thoughts on this? I'd like to get rid of my custom version of Jersey.

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

Successfully merging this pull request may close these issues.

2 participants