Wade Simmons created THRIFT-3494: ------------------------------------ Summary: Node.js should use Buffer.isBuffer Key: THRIFT-3494 URL: https://issues.apache.org/jira/browse/THRIFT-3494 Project: Thrift Issue Type: Bug Components: Node.js - Library Affects Versions: 0.9.3 Reporter: Wade Simmons
Newer versions of Node.js (such as 4.2.x) use multiple implementations for Buffer objects and you cannot just use {{instanceof Buffer}} to detect them. Node instead has provided a {{Buffer.isBuffer}} method since at least version 0.8 which should be used to detect buffers. The current {{instanceof Buffer}} method can fail like this with Node 4.2.3: {{{ Error: writeString called without a string/Buffer argument: [object ArrayBuffer] }}} Opening Github pull request with the patch shortly. -- This message was sent by Atlassian JIRA (v6.3.4#6332)