Skip to content

HTTP/2 proxy fix, CI workflow ngtcp2-linux with caches#11527

Closed
icing wants to merge 10 commits intocurl:masterfrom
icing:ci-ngtcp2-quictls-cached
Closed

HTTP/2 proxy fix, CI workflow ngtcp2-linux with caches#11527
icing wants to merge 10 commits intocurl:masterfrom
icing:ci-ngtcp2-quictls-cached

Conversation

@icing
Copy link
Contributor

@icing icing commented Jul 27, 2023

Fix in h2 proxy tunnel: progress in ingress on sending

  • depending on what is tunneled, the proxy may never get
    invoked for receiving data explicitly. Not progressing
    ingress may lead to stalls due to missed WINDOW_UPDATEs.

CI:

  • add ngtcp2 workflow, replacing the separate ngtcp2 variations
  • use cache for common nghttpx / ngtcp2 / etc build
  • use cache for building mod_h2
  • uncached time: ~32 min (longer than before since all tls libs are built)
  • cached time: ~14 min

@github-actions github-actions bot added HTTP/3 h3 or quic related CI Continuous Integration labels Jul 27, 2023
@icing icing changed the title CI ngtcp2+quictls, use nghttpx cache as in quiche build CI ngtcp2, common workflow with cache Jul 27, 2023
icing added 4 commits July 27, 2023 11:09
- depending on what is tunneled, the proxy may never get
  invoked for receiving data explicitly. Not progressing
  ingress may lead to stalls due to missed WINDOW_UPDATEs.

CI:
- add a chache for building mod_h2
@icing icing changed the title CI ngtcp2, common workflow with cache HTTP/2 proxy fix, CI workflow ngtcp2-linux with caches Jul 27, 2023
@github-actions github-actions bot added the tests label Jul 27, 2023
@icing
Copy link
Contributor Author

icing commented Jul 27, 2023

Ready for merge now.

@bagder bagder closed this in a05cce7 Jul 27, 2023
ptitSeb pushed a commit to wasix-org/curl that referenced this pull request Sep 25, 2023
- depending on what is tunneled, the proxy may never get invoked for
  receiving data explicitly. Not progressing ingress may lead to stalls
  due to missed WINDOW_UPDATEs.

CI:
- add a chache for building mod_h2

Closes curl#11527
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Continuous Integration HTTP/3 h3 or quic related tests

Development

Successfully merging this pull request may close these issues.

2 participants