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

crash if try to send after connection lost #13

Open
ghost opened this issue Jun 30, 2014 · 5 comments
Open

crash if try to send after connection lost #13

ghost opened this issue Jun 30, 2014 · 5 comments

Comments

@ghost
Copy link

ghost commented Jun 30, 2014

if you try to send a message throug a PushSocket after the Pull-Client disconnected the whole application crashes with no exception thowen.

@mhowlett
Copy link
Owner

The nanomsg c library that NNanomsg wraps aborts the process in some situations, including this one I believe, and I'm not aware of anything that can be done at the NNanomsg level to handle it. I tried to raise this in the nanomsg mailing list, but didn't get anywhere. This behavior also causes problems for me.

@kwpatrick
Copy link
Collaborator

Is there a corresponding nanomsg bug open?

Matt, did you have a workaround? If not we might consider marking the push
pull as deprecated until the behavior improves.
On Jun 30, 2014 10:46 AM, "Matt Howlett" [email protected] wrote:

The nanomsg c library that NNanomsg wraps aborts the process in some
situations, including this one I believe, and I'm not aware of anything
that can be done at the NNanomsg level to handle it. I tried to raise this
in the nanomsg mailing list, but didn't get anywhere. This behavior also
causes problems for me.


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

@mhowlett
Copy link
Owner

This was a general comment about the use of nn_assert in many places in the nanomsg, which I was guessing is the problem here.

nn_assert calls the stdlib function abort which sends SIGABRT.

when I was looking before I thought I deduced a call to nn_assert was not catchable. however this page: http://www.cplusplus.com/reference/cstdlib/abort/ includes the text "... if uncaught ..." implying it is possible.

I will look in to this soonish if no resolution - the issue affects me also - but super busy ATM with something else.

@mhowlett
Copy link
Owner

@ghost
Copy link
Author

ghost commented Jul 7, 2014

seems to be solved after update to 0.4-beta

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