Mirroristas/kaspad
2
0
Fork 0
mirror of https://github.com/kaspanet/kaspad.git synced 2025-06-07 06:36:46 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix the RPCClient leaking connections (#1820)

Browse Source 
* Fix the RPCClient leaking connections.

* Wrap the error return from GetInfo.
This commit is contained in:
stasatdaglabs 2021-08-16 15:59:35 +03:00 committed by GitHub
parent ce17348175
commit 65b5a080e4
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 18 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
infrastructure/network/rpcclient/rpcclient.go
View File

@ -39,18 +39,6 @@ func NewRPCClient(rpcAddress string) (*RPCClient, error) {
return nil, err
}
getInfoResponse, err := rpcClient.GetInfo()
if err != nil {
return nil, errors.Errorf("error making GetInfo request")
}
localVersion := version.Version()
remoteVersion := getInfoResponse.ServerVersion
if localVersion != remoteVersion {
return nil, errors.Errorf("Server version mismatch, expect: %s, got: %s", localVersion, remoteVersion)
}
return rpcClient, nil
}
@ -73,11 +61,23 @@ func (c *RPCClient) connect() error {
c.rpcRouter = rpcRouter
log.Infof("Connected to %s", c.rpcAddress)
getInfoResponse, err := c.GetInfo()
if err != nil {
return errors.Wrapf(err, "error making GetInfo request")
}
localVersion := version.Version()
remoteVersion := getInfoResponse.ServerVersion
if localVersion != remoteVersion {
return errors.Errorf("Server version mismatch, expect: %s, got: %s", localVersion, remoteVersion)
}
return nil
}
func (c *RPCClient) disconnect() error {
c.rpcRouter.router.Close()
err := c.GRPCClient.Disconnect()
if err != nil {
return err
@ -129,8 +129,12 @@ func (c *RPCClient) Reconnect() error {
func (c *RPCClient) handleClientDisconnected() {
atomic.StoreUint32(&c.isConnected, 0)
if atomic.LoadUint32(&c.isClosed) == 0 {
err := c.disconnect()
if err != nil {
panic(err)
}
c.lastDisconnectedTime = time.Now()
err := c.Reconnect()
err = c.Reconnect()
if err != nil {
panic(err)
}