[THRIFT-1665] TBinaryProtocol: exceeded message length raises generic TException

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.8
• Fix Version/s: 0.9
• Component/s: Java - Library
• Assignee: Tom Hobbs
• Reporter: Tom Hobbs
• Created: 30/Jul/12 17:30
• Updated: 11/Sep/12 18:34
• Resolved: 11/Sep/12 18:34

Description

TBinaryProtocol allows you to set a readLength attribute when creating a Factory (at least in the Java lib). In TBinaryProtocol.checkReadLength(), if the message length exceeds this, a TException is raised.

This failure should result in a TProtocolException instead to make it possible to distinguish some of the error cases. (In particular, it would make it easier for the server to notify the client that it should not reattempt sending the same message.)