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

Fix for memory leak in NATS #1217

Merged
merged 1 commit into from
Jun 3, 2022
Merged

Fix for memory leak in NATS #1217

merged 1 commit into from
Jun 3, 2022

Conversation

sungam3r
Copy link
Collaborator

@sungam3r sungam3r commented Jun 2, 2022

Fixes #1216

@sungam3r sungam3r self-assigned this Jun 2, 2022
@sungam3r sungam3r added the bugfix label Jun 2, 2022
superzer0
superzer0 reviewed Jun 3, 2022
View reviewed changes
Copy link

@superzer0 superzer0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the fix, we observed it just yesterday as well.

@sungam3r sungam3r merged commit 932e036 into master Jun 3, 2022
@sungam3r sungam3r deleted the nats branch June 3, 2022 20:40
@sungam3r
Copy link
Collaborator Author

sungam3r commented Jun 3, 2022

https://www.nuget.org/packages/AspNetCore.HealthChecks.Nats/6.0.3 Ping me back please to verify the fix.

@superzer0
Copy link

https://www.nuget.org/packages/AspNetCore.HealthChecks.Nats/6.0.3 Ping me back please to verify the fix.

Works ok for us. Thank you

@sungam3r sungam3r mentioned this pull request Jul 30, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@superzer0 superzer0 superzer0 left review comments

Assignees

@sungam3r sungam3r

Labels
bugfix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Memory Leak in NATS check (AspNetCore.HealthChecks.Nats)
2 participants
@sungam3r @superzer0