Skip to content

Conversation

maradini77
Copy link

Description

  • Summary: Replace context.TODO() with a bounded context when sending the goodbye message during peer disconnection.
  • Motivation: Unbounded calls can hang under network issues, degrading shutdown responsiveness and operational stability.
  • Change: Use context.WithTimeout(context.Background(), 5s) for goodByeFunc in disconnect path; add a brief comment.
  • Impact: More deterministic disconnect behavior; no protocol or API changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant