TUN-8848: Don't treat connection shutdown as an error condition when RPC server is done

chungthuang · chungthuang · commit ac34f94d4232 · 2025-01-09T10:07:12.000-06:00
diff --git a/tunnelrpc/quic/session_server.go b/tunnelrpc/quic/session_server.go
@@ -51,7 +51,7 @@ func (s *SessionManagerServer) Serve(ctx context.Context, stream io.ReadWriteClo
 
 	select {
 	case <-rpcConn.Done():
-		return rpcConn.Err()
+		return nil
 	case <-ctx.Done():
 		return ctx.Err()
 	}

Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ func (s *SessionManagerServer) Serve(ctx context.Context, stream io.ReadWriteClo`
`51`	`51`
`52`	`52`	`select {`
`53`	`53`	`case <-rpcConn.Done():`
`54`		`- return rpcConn.Err()`
	`54`	`+ return nil`
`55`	`55`	`case <-ctx.Done():`
`56`	`56`	`return ctx.Err()`
`57`	`57`	`}`