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

http2+transfer+gnutls, upload eos/shutdown handling #14253

Closed
wants to merge 3 commits into from

Conversation

icing
Copy link
Contributor

@icing icing commented Jul 22, 2024

http2, upload eos handling

  • replace the counting of upload lengths with the new eos send flag
  • improve frequency of stream draining to happen less on events where it is not needed
  • this PR is based on send eos flag #14220

test case improvement:

  • let client 'upload-pausing' handle http versions

transfer: force send/recv while shutdown of a direction is in progress.

gnutls: fix handling when sending the close notify blocks.

With these changes, the pytest cases work when simulating a 90% chance of socket write block.

@github-actions github-actions bot added the tests label Jul 22, 2024
@icing icing added HTTP/2 feature-window A merge of this requires an open feature window labels Jul 23, 2024
@bagder
Copy link
Member

bagder commented Aug 3, 2024

This PR now has merge conflicts.

- replace the counting of upload lengths with the new eos
  send flag
- improve frequency of stream draining to happen less on
  events where it is not needed
- this PR is based on curl#14220

http2, cf-h2-proxy: fix EAGAINed out buffer
- in adjust pollset and shutdown handling, a non-empty `ctx->outbufq`
  must trigger send polling, irregardless of http/2 flow control
- in http2, fix retry handling of blocked GOAWAY frame

test case improvement:
- let client 'upload-pausing' handle http versions
Solves the problem of blocked sends during connect phase, observed
in a HTTP/2 proxy setup.
a transfer direction.

Fix gnutls shutdown handling when close notice sending was blocked.
@icing icing changed the title http2, upload eos handling http2+transfer+gnutls, upload eos/shutdown handling Aug 4, 2024
@bagder bagder closed this in 35bf766 Aug 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-window A merge of this requires an open feature window HTTP/2 tests
Development

Successfully merging this pull request may close these issues.

2 participants