Skip to content

Commit

Permalink
Merge pull request #136 from dmcgowan/backport-reset-leak
Browse files Browse the repository at this point in the history
  • Loading branch information
fuweid authored Mar 24, 2023
2 parents 0247db1 + 8977f59 commit e1f0dab
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions server.go
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ import (
"net"
"sync"
"sync/atomic"
"syscall"
"time"

"github.com/sirupsen/logrus"
Expand Down Expand Up @@ -467,14 +468,12 @@ func (c *serverConn) run(sctx context.Context) {
// branch. Basically, it means that we are no longer receiving
// requests due to a terminal error.
recvErr = nil // connection is now "closing"
if err == io.EOF || err == io.ErrUnexpectedEOF {
if err == io.EOF || err == io.ErrUnexpectedEOF || errors.Is(err, syscall.ECONNRESET) {
// The client went away and we should stop processing
// requests, so that the client connection is closed
return
}
if err != nil {
logrus.WithError(err).Error("error receiving message")
}
logrus.WithError(err).Error("error receiving message")
case <-shutdown:
return
}
Expand Down

0 comments on commit e1f0dab

Please sign in to comment.