You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@thrift.apache.org by jk...@apache.org on 2016/11/14 15:32:03 UTC
thrift git commit: THRIFT-3961 terminate the connection to the client
if an exception occurs while processing a message Client: C++ Patch: Claudius
Heine
Repository: thrift
Updated Branches:
refs/heads/master c9877fb3b -> 7656793d0
THRIFT-3961 terminate the connection to the client if an exception occurs while processing a message
Client: C++
Patch: Claudius Heine <ch...@denx.de>
This closes #1125
Project: http://git-wip-us.apache.org/repos/asf/thrift/repo
Commit: http://git-wip-us.apache.org/repos/asf/thrift/commit/7656793d
Tree: http://git-wip-us.apache.org/repos/asf/thrift/tree/7656793d
Diff: http://git-wip-us.apache.org/repos/asf/thrift/diff/7656793d
Branch: refs/heads/master
Commit: 7656793d0de87bded5e4d8c91173b066f3f5daa8
Parents: c9877fb
Author: James E. King, III <jk...@apache.org>
Authored: Mon Nov 14 10:30:05 2016 -0500
Committer: James E. King, III <jk...@apache.org>
Committed: Mon Nov 14 10:30:05 2016 -0500
----------------------------------------------------------------------
lib/cpp/src/thrift/server/TConnectedClient.cpp | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/thrift/blob/7656793d/lib/cpp/src/thrift/server/TConnectedClient.cpp
----------------------------------------------------------------------
diff --git a/lib/cpp/src/thrift/server/TConnectedClient.cpp b/lib/cpp/src/thrift/server/TConnectedClient.cpp
index 9583284..889c885 100644
--- a/lib/cpp/src/thrift/server/TConnectedClient.cpp
+++ b/lib/cpp/src/thrift/server/TConnectedClient.cpp
@@ -84,7 +84,8 @@ void TConnectedClient::run() {
} catch (const TException& tex) {
string errStr = string("TConnectedClient processing exception: ") + tex.what();
GlobalOutput(errStr.c_str());
- // Continue processing
+ // Disconnect from client, because we could not process the message.
+ done = true;
}
}