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>
(cherry picked from commit 202c291)
  • Loading branch information
yihuang authored and mergify[bot] committed Sep 6, 2023
1 parent b80ea56 commit aec9411
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 @@ -277,7 +276,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 @@ -375,7 +374,6 @@ func startCmtNode(
cleanupFn = func() {
if tmNode != nil && tmNode.IsRunning() {
_ = tmNode.Stop()
_ = app.Close()
}
}

Expand Down

0 comments on commit aec9411

Please sign in to comment.