Skip to content

[ISS-2462] fix(redis): list subscriber consume/stop race - #2531

Merged
Lancetnik merged 12 commits into
ag2ai:mainfrom
NelsonNotes:fix/iss-2462
Oct 4, 2025
Merged

Lancetnik merged 12 commits into
ag2ai:mainfrom
NelsonNotes:fix/iss-2462

Conversation

@NelsonNotes

@NelsonNotes NelsonNotes commented Sep 30, 2025 •

Copy link
Copy Markdown
Collaborator

Description

Fixes redis ListSubscriber non-syncronized gap between message reading and handling which may lead to race condition in case of subscriber stopping.

Fixes #2462

Type of change

Bug fix (a non-breaking change that resolves an issue)

Checklist

  • My code adheres to the style guidelines of this project (just lint shows no errors)
  • I have conducted a self-review of my own code
  • I have made the necessary changes to the documentation
  • My changes do not generate any new warnings
  • I have added tests to validate the effectiveness of my fix or the functionality of my new feature
  • Both new and existing unit tests pass successfully on my local environment by running just test-coverage
  • I have ensured that static analysis tests are passing by running just static-analysis
  • I have included code examples to illustrate the modifications

@github-actions github-actions Bot added the Redis Issues related to `faststream.redis` module and Redis features label Sep 30, 2025
Comment thread docker-compose.yaml Outdated
Comment thread justfile Outdated
@Lancetnik
Lancetnik marked this pull request as ready for review October 3, 2025 06:52
@github-actions github-actions Bot added the dependencies Pull requests that update a dependency file label Oct 4, 2025
@Lancetnik
Lancetnik enabled auto-merge October 4, 2025 13:52
@Lancetnik
Lancetnik added this pull request to the merge queue Oct 4, 2025
Merged via the queue into ag2ai:main with commit e91a348 Oct 4, 2025
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file Redis Issues related to `faststream.redis` module and Redis features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: Redis ListSubscriber discards unprocessed messages on shutdown

2 participants