You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@thrift.apache.org by je...@apache.org on 2016/09/19 20:40:27 UTC

thrift git commit: THRIFT-3931 TSimpleServer: If process request encounter UNKNOWN_METHOD, don't close transport. Client: Go Patch: zhangxin

Repository: thrift
Updated Branches:
  refs/heads/master 205dc1955 -> 54f49f863


THRIFT-3931 TSimpleServer: If process request encounter UNKNOWN_METHOD, don't close transport.
Client: Go
Patch: zhangxin <xi...@ele.me>

This closes #1090


Project: http://git-wip-us.apache.org/repos/asf/thrift/repo
Commit: http://git-wip-us.apache.org/repos/asf/thrift/commit/54f49f86
Tree: http://git-wip-us.apache.org/repos/asf/thrift/tree/54f49f86
Diff: http://git-wip-us.apache.org/repos/asf/thrift/diff/54f49f86

Branch: refs/heads/master
Commit: 54f49f863567fa6fdae41d210df87ce6cec4014b
Parents: 205dc19
Author: zhangxin <xi...@ele.me>
Authored: Mon Sep 19 12:17:20 2016 +0800
Committer: Jens Geyer <je...@apache.org>
Committed: Mon Sep 19 22:37:19 2016 +0200

----------------------------------------------------------------------
 lib/go/thrift/simple_server.go | 3 +++
 1 file changed, 3 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/thrift/blob/54f49f86/lib/go/thrift/simple_server.go
----------------------------------------------------------------------
diff --git a/lib/go/thrift/simple_server.go b/lib/go/thrift/simple_server.go
index 6b3811e..8c56a4f 100644
--- a/lib/go/thrift/simple_server.go
+++ b/lib/go/thrift/simple_server.go
@@ -183,6 +183,9 @@ func (p *TSimpleServer) processRequests(client TTransport) error {
 			log.Printf("error processing request: %s", err)
 			return err
 		}
+		if err, ok := err.(TApplicationException); ok && err.TypeId() == UNKNOWN_METHOD {
+			continue
+		}
 		if !ok {
 			break
 		}