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

Problem: zyre lacks security #533

Closed
Asmod4n opened this issue Jun 5, 2017 · 4 comments
Closed

Problem: zyre lacks security #533

Asmod4n opened this issue Jun 5, 2017 · 4 comments

Comments

@Asmod4n
Copy link
Contributor

Asmod4n commented Jun 5, 2017

At the moment, anyone can impersonate a peer as he likes.
This could be solved by increasing the UUID size to 32 octets so it could be the public key of a curve key pair.
What do you think?

@bluca
Copy link
Member

bluca commented Jun 5, 2017

I think the proper solution is to add full curve support: #494

@Asmod4n
Copy link
Contributor Author

Asmod4n commented Jun 5, 2017

Ops, didn't see that.
Closed via #494

@Asmod4n Asmod4n closed this as completed Jun 5, 2017
@wesyoung
Copy link
Contributor

wesyoung commented Jun 5, 2017

i see the uuid as more like the MAC address (eg: arp). in addition to adding curve to the sockets, the easier work-around right now is probably just adding messaging signing to your application stack(?).

@Asmod4n
Copy link
Contributor Author

Asmod4n commented Jun 5, 2017

@wesyoung
I am currently only using zyre to find peers to connect to which are in the same group, i am not using zyre to send messages.
I am doing messaging with another set of sockets, which is authenticated and encrypted.

I would like to do the same with zyre alone, or else i could just reimplement the UDP broadcast messages without a need to link my app to czmq and zyre, which is a bit complicated when you want to cross compile.

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

3 participants