Re: Error with binary data with node.js

2014-10-15 Thread Randy Abernethy
The source at RandyAbernethy/thriftnode is a copy of https://github.com/apache/thrift/tree/master/lib/nodejs. You can easily make your own copy but you can not just fork the apache/thrift repo (which is why I maintain a node only repo). The semver tags on the apache repo will cause npm to use code

Re: Error with binary data with node.js

2014-10-15 Thread Rodolfo Ochoa
where is this generated? how can I generate my own? On Sat, Sep 20, 2014 at 6:03 PM, Randy Abernethy wrote: > I think the official answer to that is always "real soon now". > > In the mean time you can get the current master with npm as follows: > > $ npm install RandyAbernethy/thriftnode > > Th

Re: Error with binary data with node.js

2014-09-20 Thread Randy Abernethy
I think the official answer to that is always "real soon now". In the mean time you can get the current master with npm as follows: $ npm install RandyAbernethy/thriftnode This is a clone of the thrift master branch setup to work with npm. On Fri, Sep 19, 2014 at 4:41 PM, Rodolfo Ochoa wrote:

Re: Error with binary data with node.js

2014-09-19 Thread Rodolfo Ochoa
Oh, it is fixed on newest version... do you have a release date? thanks On Fri, Sep 19, 2014 at 6:24 PM, Rodolfo Ochoa wrote: > this error start happening here: > https://issues.apache.org/jira/browse/THRIFT-1679 > which is a correct fix, but then you need to correctly implement binary. > > -R

Re: Error with binary data with node.js

2014-09-19 Thread Rodolfo Ochoa
this error start happening here: https://issues.apache.org/jira/browse/THRIFT-1679 which is a correct fix, but then you need to correctly implement binary. -Rod.O On Fri, Sep 19, 2014 at 5:33 PM, Rodolfo Ochoa wrote: > the problem is... > the problem is that for the generated code, > thrift map

Re: Error with binary data with node.js

2014-09-19 Thread Rodolfo Ochoa
the problem is... the problem is that for the generated code, thrift maps binary data into strings, and uses the method readString from the BinaryProtocol protocol.js:280 TBinaryProtocol.prototype.readString = function() { instead of using readBinary, protocol.js:275 TBinaryProtocol.prototype.readB

Re: Error with binary data with node.js

2014-09-18 Thread Randy Abernethy
Hey Rodolfo, While I can not say for sure not having seen your code, I would be highly suspect of any Thift node lib prior to 0.9.2, which is not yet released. Try the current master, it has too many fixes to mention and will likely solve your problem. You don't need to change versions on any of t

Re: Error with binary data with node.js

2014-09-18 Thread Rodolfo Ochoa
on node: thrift@0.9.1 on java: libthrift-0.9.1.jar On Thu, Sep 18, 2014 at 9:09 PM, Randy Abernethy wrote: > What version of thrift are you using? > On Sep 18, 2014 7:46 PM, "Rodolfo Ochoa" wrote: > > > I have a Java server sending info to C/Python/node.js clients, > > I don't have issues wi

Re: Error with binary data with node.js

2014-09-18 Thread Randy Abernethy
What version of thrift are you using? On Sep 18, 2014 7:46 PM, "Rodolfo Ochoa" wrote: > I have a Java server sending info to C/Python/node.js clients, > I don't have issues with C or Python, only with node.js > on Java side, I'm sending a byte array, and if I print the values of this > byte array

Error with binary data with node.js

2014-09-18 Thread Rodolfo Ochoa
I have a Java server sending info to C/Python/node.js clients, I don't have issues with C or Python, only with node.js on Java side, I'm sending a byte array, and if I print the values of this byte array I got: byte codes: -84 -19 0 5 119 44 0 5 -50 108 0 44 75 73 78 -120 -103 -59 -124 3 -92 -121