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

[HYP-39] request size did not match content length #24

Open
AvinashUtekar opened this issue Sep 21, 2023 · 7 comments
Open

[HYP-39] request size did not match content length #24

AvinashUtekar opened this issue Sep 21, 2023 · 7 comments
Assignees
Labels
bug Something isn't working linear Created by Linear-GitHub Sync

Comments

@AvinashUtekar
Copy link

AvinashUtekar commented Sep 21, 2023

image

i have added hyperdx in nodejs project now i am getting this error on api call , specifically for PUT,POST requests

HYP-39

@MikeShi42
Copy link
Contributor

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?

@michaelsharpe
Copy link

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.

@MikeShi42
Copy link
Contributor

@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 advancedNetworkCapture. Thank you so much for the extra info, and apologies for making you have to hunt around to figure out this issue.

@MikeShi42 MikeShi42 added the linear Created by Linear-GitHub Sync label Sep 29, 2023
@MikeShi42 MikeShi42 changed the title request size did not match content length [HYP-39] request size did not match content length Sep 29, 2023
@MikeShi42 MikeShi42 added the bug Something isn't working label Sep 29, 2023
@michaelsharpe
Copy link

@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 :)

@wrn14897 wrn14897 self-assigned this Oct 9, 2023
@MikeShi42
Copy link
Contributor

@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.

@wrn14897
Copy link
Contributor

wrn14897 commented Dec 21, 2023

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 :)

@michaelsharpe
Copy link

@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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working linear Created by Linear-GitHub Sync
Projects
None yet
Development

No branches or pull requests

4 participants