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

IPv6 link local addresses are not handled properly #1661

Closed
syenna opened this issue Dec 10, 2015 · 5 comments
Closed

IPv6 link local addresses are not handled properly #1661

syenna opened this issue Dec 10, 2015 · 5 comments

Comments

@syenna
Copy link
Contributor

syenna commented Dec 10, 2015

ZeroMQ library is not handling the IPv6 link local addresses properly when creating sockets. IPv6 link local addresses include % followed by an interface name. This is not considered by the ZeroMQ library in handling the socket address passed. tcp_address.cpp and socket_base.cpp need to be modified to solve this issue.

@syenna
Copy link
Contributor Author

syenna commented Dec 10, 2015

Pull request #1659 fixes this issue. It is already merged on to the libzmq master branch. This fix needs to be backported to the stable branch zeromq4-1.

@syenna syenna closed this as completed Dec 10, 2015
@hintjens
Copy link
Member

Backported to 4-1 stable, thanks. :)

On Thu, Dec 10, 2015 at 5:04 AM, Sathish Yenna notifications@github.com
wrote:

Closed #1661 #1661.


Reply to this email directly or view it on GitHub
#1661 (comment).

@syenna
Copy link
Contributor Author

syenna commented Dec 10, 2015

I do not see the changes on 4-1 stable yet. Am I missing something here? Please let me know.

@hintjens
Copy link
Member

Ah, sorry, didn't merge yet... hang on... :-) ok, done.

On Thu, Dec 10, 2015 at 4:58 PM, Sathish Yenna notifications@github.com
wrote:

I do not see the changes on 4-1 stable yet. Am I missing something here?
Please let me know.


Reply to this email directly or view it on GitHub
#1661 (comment).

@syenna
Copy link
Contributor Author

syenna commented Dec 10, 2015

Thanks!
Next comes changes for czmq... I will work on them.

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

No branches or pull requests

2 participants