[ https://issues.apache.org/jira/browse/THRIFT-1267?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Hans Duedal updated THRIFT-1267: -------------------------------- Attachment: thrift-1267-callback.patch New patch, with callback(err, result) instead. Updated gist:1151782 usecase also. Should support syntax from gist:1098395 (Russel Haering) as well. Implemented it using instanceof and early returns. What do you think? > Node.js can't throw exceptions. > -------------------------------- > > Key: THRIFT-1267 > URL: https://issues.apache.org/jira/browse/THRIFT-1267 > Project: Thrift > Issue Type: Improvement > Components: JavaScript - Compiler > Affects Versions: 0.7 > Reporter: Hans Duedal > Labels: compiler, javascript, node, nodejs > Fix For: 0.8 > > Attachments: nodejs-exception.patch, thrift-1267-callback.patch > > > There is no way as far as I can tell for node.js servers to throw thrift > exceptions. > I have made a patch to allow it to throw exceptions. It lets the node.js > server implementation give params directly to the result object, thereby > being able to specify the exception. It doesn't affect normal (non exception) > return data. > Test case: https://gist.github.com/1151782 > Install thrift module "npm install thrift", generate thrift "thrift --gen > js:node test.thrift" and run server then client. -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira