Skip to content

tests: add test cases for http2_subreq_error_wakeup.patch#2464

Merged
zhuizhuhaomeng merged 1 commit intoopenresty:masterfrom
tzssangglass:http2_subreq_error_wakeup
May 9, 2026
Merged

tests: add test cases for http2_subreq_error_wakeup.patch#2464
zhuizhuhaomeng merged 1 commit intoopenresty:masterfrom
tzssangglass:http2_subreq_error_wakeup

Conversation

@tzssangglass
Copy link
Copy Markdown
Contributor

I hereby granted the copyright of the changes in this pull request
to the authors of this lua-nginx-module project.

sister PR of: openresty/openresty#1083

@tzssangglass
Copy link
Copy Markdown
Contributor Author

t/189-http2-subreq-error-wakeup.t failed without fix: https://github.com/openresty/lua-nginx-module/pull/2464/checks?check_run_id=54450070301

I would rerun it with fixed openresty later

@tzssangglass tzssangglass force-pushed the http2_subreq_error_wakeup branch from 13dd115 to 83161e4 Compare November 5, 2025 07:58
@tzssangglass
Copy link
Copy Markdown
Contributor Author

Hi @zhuizhuhaomeng , pls help to rerun CI(Job has been terminated due to insufficient credits balance)

@tzssangglass
Copy link
Copy Markdown
Contributor Author

Hi @zhuizhuhaomeng, could you help me move forward with this fix?

@zhuizhuhaomeng
Copy link
Copy Markdown
Contributor

We need some test cases for this PR.

@zhuizhuhaomeng
Copy link
Copy Markdown
Contributor

OK, I see it is the test case itself.
I will review it ASAP.

Signed-off-by: tzssangglass <tzssangglass@gmail.com>
Copilot AI review requested due to automatic review settings May 7, 2026 04:49
@zhuizhuhaomeng zhuizhuhaomeng force-pushed the http2_subreq_error_wakeup branch from 83161e4 to dad5b90 Compare May 7, 2026 04:49
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review is ineligible. To be eligible to request a review, you need a paid Copilot license, or your organization must enable Copilot code review.

@zhuizhuhaomeng zhuizhuhaomeng merged commit 7873a51 into openresty:master May 9, 2026
7 of 11 checks passed
@tzssangglass tzssangglass deleted the http2_subreq_error_wakeup branch May 9, 2026 03:53
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.

3 participants