michaelwittig / node-q

Q interfacing with Node.js
MIT License
52 stars 14 forks source link

Only recognize sync replies for request/response answeres #2

Closed michaelwittig closed 9 years ago

michaelwittig commented 9 years ago

Reported from Markus Sieber via personal-kdbplus@googlegroups.com

i think you have a race condition in your message dispatcher: assume you are sending a sync message from node. But while you do so, kdb sends you an upd. in your code the activeRequestResponse is set to true. The solution is to correctly check the ipc header (see here: http://code.kx.com/wiki/Reference/ipcprotocol). I think you can go for the message type