Skip to content

feat(build-curl-h3): auto-trigger on tag push - #1

Closed
tzssangglass wants to merge 1 commit into
openresty:mainfrom
tzssangglass:build-curl-h3-workflow
Closed

feat(build-curl-h3): auto-trigger on tag push#1
tzssangglass wants to merge 1 commit into
openresty:mainfrom
tzssangglass:build-curl-h3-workflow

Conversation

@tzssangglass

@tzssangglass tzssangglass commented Jun 14, 2026

Copy link
Copy Markdown

During the process of submitting a PR for lua-nginx-module earlier, I encountered a CI failure: https://github.com/openresty/lua-nginx-module/actions/runs/26863098901/job/79220859392

After conducting an investigation, I found that it was due to

TEST 2 (lua_need_request_body off) in HTTP/3 mode, curl times out:

WARNING: t/031-post-args.t TEST 2: lua_need_request_body off - command "curl -i -H User-Agent:
-H Accept: -H Host: -sS --http3-only -k -X POST -H Host: localhost -H Content-Length: 9
-H Content-Type:  --data-binary @.../curl.data.bin --connect-timeout 3 --max-time 3
https://127.0.0.1:1984/lua" generates stderr output:
curl: (28) Operation timed out after 3000 milliseconds with 187 out of 187 bytes received

Key information: curl received the full 187-byte 500 response but waited until the --max-time 3-second timeout expired before exiting.
Subsequently, the IPC::Run timer also timed out (in the CI high-load environment, it triggered before curl), directly terminating the test process and causing the subsequent 8 tests to fail, changing the plan from 52 to 44.

Same issue on curl repo:

curl fix it at 8.5.0: curl/curl@9cc5787

The current version of this repo release is 8.3.0

So I plan to make this repo release curl 8.20.0 to meet the testing requirements of the lua-nginx-module.

My release test on this PR: https://github.com/tzssangglass/openresty-deps-prebuild/releases/tag/v1.0.1

I hereby granted the copyright of the changes in this pull request
to the authors of this openresty-deps-prebuild project.

@tzssangglass
tzssangglass marked this pull request as draft June 14, 2026 15:37
@tzssangglass
tzssangglass force-pushed the build-curl-h3-workflow branch from 9d321c9 to 9d84ab5 Compare June 14, 2026 16:32
@tzssangglass
tzssangglass force-pushed the build-curl-h3-workflow branch from 9d84ab5 to 598cf67 Compare June 14, 2026 16:42
@tzssangglass
tzssangglass marked this pull request as ready for review June 14, 2026 17:02
@tzssangglass tzssangglass changed the title feat(build-curl-h3): auto-trigger on tag push and release publish feat(build-curl-h3): auto-trigger on tag push Jun 14, 2026
@tzssangglass

Copy link
Copy Markdown
Author

@zhuizhuhaomeng

Copy link
Copy Markdown
Contributor

We can use the curl directly from https://github.com/stunnel/static-curl

@tzssangglass

Copy link
Copy Markdown
Author

We can use the curl directly from stunnel/static-curl

ok, openresty/lua-nginx-module#2509

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

Successfully merging this pull request may close these issues.

2 participants