dialers/retrier: avoid retrying on successful rw

ignoramous · ignoramous · commit 545f1a686ed7 · 2025-05-15T22:00:15.000+05:30
diff --git a/intra/dialers/retrier.go b/intra/dialers/retrier.go
@@ -425,6 +425,8 @@ func (r *retrier) Read(buf []byte) (n int, err error) {
 				c = r.conn // re-assign c to newConn, if any; may be nil
 				if c == nil || core.IsNil(c) {
 					err = core.UniqErr(err, retryerr)
+				} else {
+					err = nil
 				}
 				logeor(retryerr, log.I)("retrier: read#%d + (mult? %t / c: %d): [%s<=%s] %d; err? %v",
 					r.retryCount, r.multidial, r.nextDialerIdx, laddr(c), r.raddr, n, retryerr)

Original file line number	Diff line number	Diff line change
`@@ -425,6 +425,8 @@ func (r *retrier) Read(buf []byte) (n int, err error) {`
`425`	`425`	`c = r.conn // re-assign c to newConn, if any; may be nil`
`426`	`426`	`if c == nil \|\| core.IsNil(c) {`
`427`	`427`	`err = core.UniqErr(err, retryerr)`
	`428`	`+ } else {`
	`429`	`+ err = nil`
`428`	`430`	`}`
`429`	`431`	`logeor(retryerr, log.I)("retrier: read#%d + (mult? %t / c: %d): [%s<=%s] %d; err? %v",`
`430`	`432`	`r.retryCount, r.multidial, r.nextDialerIdx, laddr(c), r.raddr, n, retryerr)`