Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Nats Jetstream: consuming a message without subscription causes deadlock in the consumer #206

Closed
mymdz opened this issue Apr 29, 2024 · 2 comments
Closed

Nats Jetstream: consuming a message without subscription causes deadlock in the consumer #206

mymdz opened this issue Apr 29, 2024 · 2 comments
Assignees
@lerenn @mymdz
Labels
bug Something isn't working

Comments

@mymdz
Copy link
Contributor

mymdz commented Apr 29, 2024

Steps to reproduce:

  1. Generate multiple operations: A and B
  2. Subscribe to a specific subject (A) using UserController, without UserSubscriber implementation
  3. Send a message to B. A warning will be printed: Received message for not subscribed channel 'B'. Message will be ack'd.
  4. Send a message to A - nothing happens

There is a deadlock because of writing to nil channel
@mymdz mymdz mentioned this issue Apr 29, 2024
Merged
@mymdz
Copy link
Contributor Author

mymdz commented Apr 29, 2024

Fixed with #207

@lerenn lerenn assigned lerenn and mymdz and unassigned lerenn Apr 30, 2024
@lerenn lerenn added the bug Something isn't working label Apr 30, 2024
@lerenn lerenn self-assigned this Apr 30, 2024
@lerenn
Copy link
Owner

lerenn commented Apr 30, 2024

Hello @mymdz ! Good jobs ! Thanks for taking the time to report the bug AND to bring a fix ! ☺️
I'll review it ASAP and I'll merge it (maybe not today, but tomorrow).

@lerenn lerenn closed this as completed May 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lerenn lerenn

@mymdz mymdz

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lerenn @mymdz