Skip to content

Commit bc66da9

Browse files
authored
Merge pull request #1570 from carver/dangling-request-future
Resolve dangling future in ResponseCandidateStream
2 parents 6ccff63 + 8e65d28 commit bc66da9

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

trinity/protocol/common/managers.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -206,6 +206,12 @@ async def _cleanup(self) -> None:
206206
self,
207207
)
208208

209+
def __del__(self) -> None:
210+
if self.pending_request is not None:
211+
_, future = self.pending_request
212+
if future.cancel():
213+
self.logger.debug("Forcefully cancelled a pending response in %s", self)
214+
209215
def deregister_peer(self, peer: BasePeer) -> None:
210216
if self.pending_request is not None:
211217
self.logger.debug("Peer stream %r shutting down, cancelling the pending request", self)

0 commit comments

Comments
 (0)