gsadmin/ziti
1
0
Fork 0
mirror of https://github.com/openziti/ziti.git synced 2026-10-12 23:32:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

Don't log retransmission error if the associated xgress is closed

Browse Source
This commit is contained in:
Paul Lorenz
2021-01-27 10:43:41 -05:00
parent 40091b50e9
commit 38b628fa4a
1 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
router/xgress/retransmitter.go
+7 -2
View File
@@ -140,8 +140,13 @@ func (retransmitter *Retransmitter) retransmitSender() {
case retransmit := <-retransmitter.retransmitSend:
if !retransmit.isAcked() {
if err := retransmitter.forwarder.ForwardPayload(retransmit.x.address, retransmit.payload); err != nil {
logger.WithError(err).Errorf("unexpected error while retransmitting payload from %v", retransmit.x.address)
retransmissionFailures.Mark(1)
// if xgress is closed, don't log the error. We still want to try retransmitting in case we're re-sending end of session
if !retransmit.x.closed.Get() {
logger.WithError(err).Errorf("unexpected error while retransmitting payload from %v", retransmit.x.address)
retransmissionFailures.Mark(1)
} else {
logger.WithError(err).Tracef("unexpected error while retransmitting payload from %v", retransmit.x.address)
}
} else {
retransmit.markSent()
retransmissions.Mark(1)