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

HTTP transport layer hangs? #126

Closed
dnaaun opened this issue Dec 9, 2024 · 5 comments
Closed

HTTP transport layer hangs? #126

dnaaun opened this issue Dec 9, 2024 · 5 comments

Comments

@dnaaun
Copy link

dnaaun commented Dec 9, 2024

I need to use the HTTP transport because I want to test websockets, but I think the HTTP transport layer truncates the body at around 10-11KB. Is that expected?

I now have a new issue with the HTTP transport layer hanging, which I've outlined in the comment below.

@JosephLenton
Copy link
Owner

Hey @dnaaun , no it isn’t expected.

Can you please provide more details, and if possible a code example of the issue.

@dnaaun
Copy link
Author

dnaaun commented Dec 13, 2024

Thanks for the quick reply Joseph! I tried to create a minimal repro, but the minimal repro didn't reproduce the problem 😂 . Instead, I'm now having an issue where the HTTP transport simply hangs. I'll definitely report back if I get around to making a minimal repo. But if, in the mean time, anyone wants to look into this with a repro (tho not very minimal),

UPDATE: The below instructions don't actually work because you need some env vars set.

  1. The repo is at https://github.com/dnaaun/heimisch
  2. Running this test by running cargo test --package backend -- hook --nocapture shows the test is hanging.
  3. One can be sure that the issue has to do with http transport because one can comment out this line, and that same test invocation passes.

@dnaaun dnaaun changed the title HTTP transport layer seems to truncate the body at around 10-11KB? HTTP transport layer hangs? Dec 13, 2024
@JosephLenton
Copy link
Owner

Hey @dnaaun , I had a look today at this issue. On the topic of large payloads; I have just merged tests for WebSocket, Text Http requests, and Json Http requests, where I upload 16mb of data. Those tests have passed. I'm unable to replicate your issue.

On the topic of the tests hanging; I cannot see anything wrong with the Axum Test setup in your project. However I won't be able to run your project without rewriting large amounts of it. I'd be happy to help debug, however I'm afraid you will need to provide a more minimal example.

@dnaaun
Copy link
Author

dnaaun commented Dec 14, 2024

Merci, vielen dank, and thank you for looking into this!

you will need to provide a more minimal example.

That's beyond fair. But I'm curious:

I won't be able to run your project without rewriting large amounts of it.

Thanks for letting me know!

@dnaaun
Copy link
Author

dnaaun commented Dec 14, 2024

Closing until I provide a more minimal repro.

@dnaaun dnaaun closed this as completed Dec 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants