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

Ensure error being returned to native client #387

Merged
merged 3 commits into from
Jul 8, 2013
Merged

Ensure error being returned to native client #387

merged 3 commits into from
Jul 8, 2013

Conversation

brianc
Copy link
Owner

@brianc brianc commented Jun 30, 2013

This should fix #350.

Can you verify?

@brianc
Copy link
Owner Author

brianc commented Jun 30, 2013

Interesting...this fails on travis but passes on my machine. I'll try this again on my linux box when I get back home; maybe it will fail there.

@pevot
Copy link

pevot commented Jul 1, 2013

make test-all is ok on my Linux server.
The problem I observed i my application is olved with this fix.

@brianc
Copy link
Owner Author

brianc commented Jul 1, 2013

Thanks for your help everyone! This is one of those weird hard to track down issues for me, and your insight is really appreciated.

brianc added a commit that referenced this pull request Jul 8, 2013
Ensure error being returned to native client
@brianc brianc merged commit 9d773a2 into master Jul 8, 2013
@brianc brianc deleted the issues/350 branch July 8, 2013 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Uncatchable "Connection error."
2 participants