Skip to content

Commit

Permalink
remove duplicated app.Close call (#17619)
Browse files Browse the repository at this point in the history
Co-authored-by: Facundo Medica <14063057+facundomedica@users.noreply.github.com>
  • Loading branch information
yihuang and facundomedica authored Sep 5, 2023
1 parent 9766142 commit 202c291
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions server/start.go
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ package server

import (
"context"
"errors"
"fmt"
"io"
"net"
Expand Down Expand Up @@ -275,7 +274,7 @@ func startStandAlone(svrCtx *Context, app types.Application, opts StartCmdOption
// so we can gracefully stop the ABCI server.
<-ctx.Done()
svrCtx.Logger.Info("stopping the ABCI server...")
return errors.Join(svr.Stop(), app.Close())
return svr.Stop()
})

return g.Wait()
Expand Down Expand Up @@ -373,7 +372,6 @@ func startCmtNode(
cleanupFn = func() {
if tmNode != nil && tmNode.IsRunning() {
_ = tmNode.Stop()
_ = app.Close()
}
}

Expand Down

0 comments on commit 202c291

Please sign in to comment.