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.

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

[v9]: Don't attach response body to event automatically if maxResponseBodySize is set. currently happens in dio integration #2637

Open
buenaflor opened this issue Jan 29, 2025 · 0 comments
Assignees
Milestone

Comments

@buenaflor
Copy link
Contributor

buenaflor commented Jan 29, 2025

Description

  • we try to read the content-length header
  • if no headers present, we don't read the body because we don't know the size of it
  • otherwise if a header exists, we introduce a hard limit, maybe 0.15mb or higher, this is what javascript does for replay and read the body if it is under the threshold
  • if it exceeds the hard limit we don't read the body

The read body is then attached to the hints in beforeSend and beforeSendTransaction and the user can then query it themselves if it is available

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Todo
Development

No branches or pull requests

2 participants