[ https://issues.apache.org/jira/browse/THRIFT-4639?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16629935#comment-16629935 ]
ASF GitHub Bot commented on THRIFT-4639: ---------------------------------------- phlundblom commented on issue #1597: THRIFT-4639: Use correct sequence number for multiplexed protocol URL: https://github.com/apache/thrift/pull/1597#issuecomment-425005337 How does one run the NodeJS tests? Unfortunately I don't find it obvious from the Makefiles and/or the Travis outputs ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Sequence numbering for multiplexed protocol broken > -------------------------------------------------- > > Key: THRIFT-4639 > URL: https://issues.apache.org/jira/browse/THRIFT-4639 > Project: Thrift > Issue Type: Bug > Components: Node.js - Library > Affects Versions: 0.11.0 > Reporter: PH Lundblom > Priority: Blocker > > Handling of client sequence numbering for multiplexed protocol is broken. > Current implementation uses client internal variable "seqid" which should be > "_seqid" > -- This message was sent by Atlassian JIRA (v7.6.3#76005)