SockJS swallows errors in onmessage #61

Closed
majek opened this Issue Apr 21, 2012 · 1 comment

Comments

Projects
None yet
2 participants
@majek
Member

majek commented Apr 21, 2012

As suggested here: meteor/meteor@e0722be#L4R238

majek added a commit that referenced this issue Apr 24, 2012

@majek majek closed this in aa54697 Apr 24, 2012

@glasser

This comment has been minimized.

Show comment Hide comment
@glasser

glasser Sep 26, 2012

Contributor

Thanks for noticing! meteor/meteor@a016e83

Contributor

glasser commented Sep 26, 2012

Thanks for noticing! meteor/meteor@a016e83

mquandalle pushed a commit to mquandalle/meteor-accounts that referenced this issue Mar 31, 2016

Let exceptions from stream message callbacks percolate upwards.
We used to be unable to do this because of a SockJS bug, but the SockJS folks
read our XXX comment and fixed the bug!
sockjs/sockjs-client#61
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment