Skip to content

Commit a0db075

Browse files
authored
If rendering has failed due to a net.OpError stop rendering (attempt 2) (#19049)
Unfortunately #18642 does not work because a `*net.OpError` does not implement the `Is` interface to make `errors.Is` work correctly - thus leading to the irritating conclusion that a `*net.OpError` is not a `*net.OpError`. Here we keep the `errors.Is` because presumably this will be fixed at some point in the golang main source code but also we add a simply type cast to also check. Fix #18629 Signed-off-by: Andrew Thornton <[email protected]>
1 parent ba470a8 commit a0db075

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

modules/context/context.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -269,7 +269,7 @@ func (ctx *Context) ServerError(logMsg string, logErr error) {
269269
func (ctx *Context) serverErrorInternal(logMsg string, logErr error) {
270270
if logErr != nil {
271271
log.ErrorWithSkip(2, "%s: %v", logMsg, logErr)
272-
if errors.Is(logErr, &net.OpError{}) {
272+
if _, ok := logErr.(*net.OpError); ok || errors.Is(logErr, &net.OpError{}) {
273273
// This is an error within the underlying connection
274274
// and further rendering will not work so just return
275275
return

0 commit comments

Comments
 (0)