Merge pull request #535 from murDDock/dev

xujianhai666 · web-flow · commit 99c88b4f9f30 · 2020-11-26T14:19:33.000+08:00
fix(remote_client):avoid goroutine leak
diff --git a/internal/remote/remote_client.go b/internal/remote/remote_client.go
@@ -194,7 +194,7 @@ func (c *remotingClient) processCMD(cmd *RemotingCommand, r *tcpConnWrapper) {
 				responseFuture.ResponseCommand = cmd
 				responseFuture.executeInvokeCallback()
 				if responseFuture.Done != nil {
-					responseFuture.Done <- true
+					close(responseFuture.Done)
 				}
 			})
 		}

Original file line number	Diff line number	Diff line change
`@@ -194,7 +194,7 @@ func (c remotingClient) processCMD(cmd RemotingCommand, r *tcpConnWrapper) {`
`194`	`194`	`responseFuture.ResponseCommand = cmd`
`195`	`195`	`responseFuture.executeInvokeCallback()`
`196`	`196`	`if responseFuture.Done != nil {`
`197`		`- responseFuture.Done <- true`
	`197`	`+ close(responseFuture.Done)`
`198`	`198`	`}`
`199`	`199`	`})`
`200`	`200`	`}`