- TCompactProtocol.prototype.writeBool not referencing `booleanField_`
on `this` correctly. Also using `NULL` instead of `null`.
- TCompactProtocol.prototype.writeVarint64 had a typo for TProtocolException
- TCompactProtocol.prototype.readMapBegin had a typo between `kvtype` and
`kvType`
- createMultiplexServer leaked a global var `processStatus`
- TFramedTransport had a line that was accidently copy pasted, leading
to a global leak of `frameLeft`. (I created the patch the introduced
this copy paste error in Thrift-1353, so I can confirm it was
a mistake).
- createWebServer tried to check a non-existent var `route` and leaked
a global `result`
Signed-off-by: Roger Meier <roger@apache.org>
There were many bugs in the current TFramedTransport.receiver caused by
merge mistakes and bad patches. Simplify the logic so it is easier to
reason about and prevent future issues.
- THRIFT-2194 Fixed one bug with residual not being set
- THRIFT-2205 Reverted the above fix (by accident) and broke it further
by including InputBufferUnderrunError in TFramedTransport (which is
incorrect)
This patch cleans up TFramedTransport.receiver by only have one hold
over buffer instead of two (frame + residual).
Added cross tests for ruby with ruby, cpp, java, node and python.
ruby_protocols="binary compact json accel"
ruby_transports="buffered framed"
ruby_sockets="ip"
Created new TestServer.rb and TestClient.rb for this purpose
Patch: Chamila Dilshan Wijayarathna
This closes#136
----
commit f55437d55549ced475092518f432936c4e760639
Author: Pascal Bach <pascal.bach@siemens.com>
Date: 2014-04-09T09:19:18Z
TPipeServer requires OverlappedSubmissionThread.cpp|h
so include it in the corresponding VS files
commit 63a3309a83e6fb09b589a61fe56c5abe1157acb5
Author: Pascal Bach <pascal.bach@siemens.com>
Date: 2014-03-26T15:44:07Z
Replace Env variables for 3rdparty
----
Patch: Pascal Bach
Client: C#
Patch: Michael Blättler
This closes#133
commit 57494794e787356ee98229cac35ea7aaa60ad562
Author: mblaettler <michi.blaettler@bluewin.ch>
Date: 2014-06-05T11:41:05Z
THRIFT-2568: Implemented possibility to use own certificate handler
Added 'ssl' tests for python tests.
Added '-transport arg (=buffered) transport: buffered, framed, http' to test/py/TestServer.py and test/py/TestClient.py and removed '-framed' arguement.
Changed test/py/RunClientServer.py to match above changes.
Added tests to compact protocol in python cases.
Added tests to test BinaryAccelarated protocol with Binary Protocol.
Changed py/TestClient.py and py/TestServer.py from --proto to --protocol parameter
Patch: Chamila Dilshan Wijayarathna & Roger Meier
