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.

Sign up for GitHub

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

Jump to bottom

Add support for sending a request body with HTTP/2 #3302

Open
sethmlarson opened this issue Jan 22, 2024 · 1 comment
Open

Add support for sending a request body with HTTP/2 #3302

sethmlarson opened this issue Jan 22, 2024 · 1 comment
Labels
💰 Bounty $500 If you complete this issue we'll pay you $500 on OpenCollective!
Milestone
HTTP/2

Comments

@sethmlarson
Copy link
Member

  • Requests that don't have a body should set end_stream=True on the Headers event.
  • There should never be an empty Data event, some HTTP/2 implementations don't handle this properly.
  • Must respect window of server. Maybe this is handled automatically?
@sethmlarson sethmlarson added the 💰 Bounty $500 If you complete this issue we'll pay you $500 on OpenCollective! label Jan 22, 2024
@sethmlarson sethmlarson added this to the HTTP/2 milestone Jan 22, 2024
@Nightrider0098
Copy link
Contributor

Hey was working on the issue. Tried to add a checks and while loop when handling body.
#3322

Is it the correct way to handle the body, or i need to do something different.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
💰 Bounty $500 If you complete this issue we'll pay you $500 on OpenCollective!
Projects
None yet
Milestone
HTTP/2
Development

No branches or pull requests
2 participants
@sethmlarson @Nightrider0098