Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Version of sentry dependency is 2 years old, many releases behind #527

Open
matthewbjones opened this issue Feb 13, 2025 · 1 comment
Open

Version of sentry dependency is 2 years old, many releases behind #527

matthewbjones opened this issue Feb 13, 2025 · 1 comment
Labels
dependencies Pull requests that update a dependency file

Comments

@matthewbjones
Copy link

The pingora-core crate specifies the 0.26.0 version of sentry, and there have been many releases and updates to this crate over the last 2 years.

I mention this because I've been doing some memory profiling and it seems like when a request panics, Sentry does retrieve debugging information and submit to Sentry, but the memory is never deallocated. I am unsure if upgrading to the latest version of sentry will solve this issue, but there have been 24 releases of the sentry crate since 0.26.0.
@matthewbjones matthewbjones mentioned this issue Feb 13, 2025
Open
@gdavidsson gdavidsson added the dependencies Pull requests that update a dependency file label Feb 14, 2025
@matthewbjones
Copy link
Author

Whenever there is a panic thrown during an HTTP request, memory spikes as Sentry prepares & submits its report. However, it doesn't look like that memory is ever freed:
Image

Image

# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@matthewbjones @gdavidsson