[THRIFT-1352] Thrift server

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.7
• Fix Version/s: 0.9.2
• Component/s: Node.js - Library
• Labels: node nodejs
• Patch Info: Patch Available
• Assignee: Henrique Mendonca
• Reporter: Hans Duedal
• Created: 22/Sep/11 13:26
• Updated: 10/Jul/14 13:42
• Resolved: 10/Jul/14 13:42

Description

If a thrift level protocol error occurs. For instance one makes a few changes to the thrift IDL file and a client connects using the old API an unhandled exception can be thrown from the TBinaryProtocol class. By adding a try-catch to the process() call these errors can be caught, the connection closed, and an error event emitted.

Likewise if a client closes the socket, ie. due to a timeout condition, another unhandled exception is thrown from stream.write().

Patch/Pull-request: https://github.com/wadey/node-thrift/pull/16

Issue Links

• Is contained by: THRIFT-1353 Switch to performance branch, get rid of BinaryParser