-
Notifications
You must be signed in to change notification settings - Fork 192
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
[HYP-39] request size did not match content length #24
Comments
Any chance you can share a bit more where this error is being thrown, what instrumentation/SDK are you using, and any other repro steps? |
I can confirm that this is a bug. We were using the node-opentelemetry package on our open-source project Magick (https://github.com/Oneirocom/Magick) and we started getting this error message one a handful of different requests. Often ones with very large payloads, though also in some other odd places. I hunted for a few days to figure it out, and when I came by this issue, I turned off called the SDK and the bug immediately went away. For us though it only happened on heroku, though I didn't actually try running with opentelemetry locally. |
@michaelsharpe ah got it okay the backend instrumentation is causing the client to throw bad requests on specific network requests. I'm going to see if we can repro, I'm suspecting it's an issue with |
@MikeShi42 No worries and thanks so much! Any status on when this might get pushed through? Would love to get your sweet logging back up and working on my server :) |
@michaelsharpe sorry this has been so much more elusive than we thought and we've picked/set this one down a few times in the process. It just hit us- do you happen to have a repro or any repro details you could share for magick? Would be much nicer if we can just spin up your project and actually repro this issue locally. |
hi @michaelsharpe @AvinashUtekar, the issue should be fixed (hyperdxio/hyperdx-js#75). Could you try upgrading the hyper otel sdk to latest (v0.4.2) ? Once you confirm the fix then we can close the issue :) |
@wrn14897 Man I am so sorry to have dropped the ball on this! I had totally forgotten about this issue, and was setting up a fresh deploy of my application and started getting the same error. Bashed my head for a while until I remembered this. I updated my package and it is all working like a charm. You guys rock! |
i have added hyperdx in nodejs project now i am getting this error on api call , specifically for PUT,POST requests
HYP-39
The text was updated successfully, but these errors were encountered: