Skip to content

Duplicate response issues#184

Open
mapogolions wants to merge 3 commits into
anthdm:masterfrom
mapogolions:fix/duplicate-respond-issues
Open

Duplicate response issues#184
mapogolions wants to merge 3 commits into
anthdm:masterfrom
mapogolions:fix/duplicate-respond-issues

Conversation

@mapogolions

@mapogolions mapogolions commented Feb 2, 2025

Copy link
Copy Markdown
Contributor

The PR reproduces certain issues caused by responding to a request.

Scenario:

  • Actor 1 sends a message using c.Request.
  • Actor 2 receives the message from its mailbox.
  • Actor 2 processes the message.
  • Actor 2 sends a response using c.Send(ctx.Sender, ...) or c.Respond().

Issue:

If Actor 2, either due to an error or intentionally, sends multiple responses, and Actor 1 is busy and delays retrieving the result, Actor 2 may become blocked, preventing it from processing new incoming messages.

To review the problem, check out the corresponding commit.

@anthdm

anthdm commented Feb 25, 2025

Copy link
Copy Markdown
Owner

I'm looking into this. Thanks for the PR.

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.

2 participants