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

getCurlCommand fails when content type is multipart/form-data and/or null #71456

Open
billyvg opened this issue May 23, 2024 · 1 comment
Open

Comments

@billyvg
Copy link
Member

billyvg commented May 23, 2024

Environment

SaaS (https://sentry.io/)

Steps to Reproduce

N/A - this is observed from Sentry issues e.g. https://sentry.sentry.io/issues/5284365861/events/fe5fc8fc7e334456a5ebee74f8fadb5e/

Expected Result

It should render the data, though it might be tricky because it may contain file data e.g. from <input type="file" />.

Actual Result

It does not render the data

Product Area

Unknown

Link

https://sentry.sentry.io/issues/5284365861/events/fe5fc8fc7e334456a5ebee74f8fadb5e/?project=11276&referrer=previous-event&statsPeriod=30d&stream_index=0&utc=true

DSN

No response

Version

No response

@billyvg
Copy link
Member Author

billyvg commented May 23, 2024

Relevant code here which was added in #6064. The exception capturing was added in #6338

billyvg added a commit that referenced this issue May 23, 2024
We have been [capturing](#6338) this issue for almost 7 years now and have not attempted to fix it so I dont see a reason to continue capturing it. I have [created this ticket](#71456) should someone decide to pick it up in the future.This could be a metric but again, not sure theres a point.
billyvg added a commit that referenced this issue May 30, 2024
We have been [capturing](#6338)
this issue for almost 7 years now and have not attempted to fix it so I
dont see a reason to continue capturing it. I have [created this
ticket](#71456) should someone
decide to pick it up in the future. This could be a metric but I don't
think there's a point as an increase of the metric doesn't mean
something is broken, it only indicates that the feature is missing.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant