fishy opened a new pull request #2296: URL: https://github.com/apache/thrift/pull/2296
Client: go Define TConfiguration following the spec, and also move the following configurations scattered around different TTransport/TProtocol into it: - max frame size for TFramedTransport - strict read and strict write for TBinaryProtocol - proto id for THeaderTransport Also add TConfiguration support for the following and their factories: - THeaderTransport and THeaderProtocol - TBinaryProtocol - TCompactProtocol - TFramedTransport - TRichTransport Also define TConfigurationSetter interface for easier TConfiguration propagation between wrapped TTransports/TProtocols. Also add simple sanity check for TBinaryProtocol and TCompactProtocol's ReadString and ReadBinary functions. Currently it only report error if the header length is larger than MaxMessageSize configured in TConfiguration, for simplicity. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org