[THRIFT-3361] Improve C# library

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.10.0
• Component/s: C# - Compiler, C# - Library, Test Suite
• Patch Info: Patch Available
• Assignee: Nobuaki Sukegawa
• Reporter: Nobuaki Sukegawa
• Created: 28/Sep/15 23:38
• Updated: 18/Mar/16 18:02
• Resolved: 18/Mar/16 18:02

Description

Server exception handling

Servers didn't propagate exceptions in handler code to the client.

Same as THRIFT-3349 (python) and THRIFT-3335 (ruby)

TBufferedTransport

System.IO.BufferedStream inside this transport made it strongly tied to underlying transport's stream state.
More concretely, when used on top of TTLSSocket it crashed because underlying SslStream is disposed by the socket while BufferedStream kept trying to touch the disposed stream after that.

So I removed BufferedStream and made it use System.IO.MemoryBuffer instead.

• Can now work on top of TTLSSocket
• Underlying transport no longer needs to be a TStreamTransport

TFramedTransport

It was allocating new MemoryBuffer for nealy every write and read of frames.
I made it allocate only once.

JSONProtocol

• Can now handle Base64 with padding for binary fields (Fixes C# part of THRIFT-3359)

TTLSSocket

• Make client certificate optional

Test

With this patch applied, all the cross test apps should be mostly valid.

• Fix SSL setup
• Semantic return code
• Add testException and testMultiException
• Add missing assertions