dapr / components-contrib

Community driven, reusable components for distributed apps

Apache License 2.0

549 stars 481 forks source link

Resolving a weird edge case in case of a poison pill message being retried, followed by a pod restart #3532

Closed passuied closed 2 months ago

passuied commented 2 months ago

Description

Resolving a weird edge case in case of a poison pill message

Moving the session.Context().Done() case ensures the next message is NOT processed while the pod is terminating...

Issue reference

3529

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests: Hard to reproduce with tests
[x] Extended the documentation: N/A

marcduiker commented 1 month ago

@holopin-bot @passuied Thank you for your contribution!

holopin-bot[bot] commented 1 month ago

Congratulations @passuied, the maintainer of this repository has issued you a badge! Here it is: https://holopin.io/claim/cm1ukzpsz59710cmdwxfu7x1m

This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account. Or if you're new to Holopin, you can simply sign up with GitHub, which will do the trick!