[ https://issues.apache.org/jira/browse/THRIFT-3787?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15276534#comment-15276534 ]
James Reggio commented on THRIFT-3787: -------------------------------------- Oh interesting! I was not aware of that behavior. Thanks for pointing it out. The code still seems a bit odd, particularly in the sense that the number of 'error' event listeners on the underlying connection should always be one, since the connection is a private object, and we attach precisely one listener. That said, if this code generates exceptions/side-effects, I'll leave it alone. Definitely don't want to unintentionally change that behavior. I'll push an update to the branch and ping you. > Node.js Connection object doesn't handle errors correctly > --------------------------------------------------------- > > Key: THRIFT-3787 > URL: https://issues.apache.org/jira/browse/THRIFT-3787 > Project: Thrift > Issue Type: Bug > Components: Node.js - Library > Reporter: James Reggio > Assignee: Randy Abernethy > Priority: Minor > > There are a handful of operation-ordering problems in the > Connection.prototype.connection_gone() method and its friends. > See the pull request for more details. -- This message was sent by Atlassian JIRA (v6.3.4#6332)