Skip to content
This repository has been archived by the owner on Apr 24, 2020. It is now read-only.

Allows sending on a PULL socket #36

Closed
mloughran opened this issue Jun 21, 2011 · 1 comment
Closed

Allows sending on a PULL socket #36

mloughran opened this issue Jun 21, 2011 · 1 comment

Comments

@mloughran
Copy link

There is no error when I call send on a PULL socket, even though this is not allowed (http://api.zeromq.org/2-1-1:zmq-socket#toc13)

The message appears to be buffered by the node code since s.currentSendBacklog() is incremented.

Would it not make sense to raise error when calling send in this case?

@JustinTulloss
Copy link
Owner

hmm, this is interesting. We only try to send if the socket says that it's writable, but a PULL socket will never be writable, so it'll just sit. I'm ok with that behavior in general, and like you said, you should never be writing to a PULL socket, so I don't really feel the need to fix this.

Let me know if you feel otherwise.

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

No branches or pull requests

2 participants