lua-nginx-module icon indicating copy to clipboard operation
lua-nginx-module copied to clipboard

bugfix: remove http2 hardcode limitation in `ngx.location` subrequest API

Open oowl opened this issue 1 year ago • 6 comments

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

context: https://github.com/openresty/lua-nginx-module/issues/2243

oowl avatar Sep 04 '24 06:09 oowl

@oowl would you please add some test cases to cover this change.

zhuizhuhaomeng avatar Sep 06 '24 04:09 zhuizhuhaomeng

Yeah, i will. please wait for me running huge kong side integration test to verify http2 location capture case problem.

oowl avatar Sep 06 '24 04:09 oowl

@oowl any progress?

zhuizhuhaomeng avatar Oct 15 '24 11:10 zhuizhuhaomeng

This pull request is now in conflict :(

mergify[bot] avatar Oct 24 '24 05:10 mergify[bot]

@zhuizhuhaomeng Hi, I have been enable all http2 test in ci config. Can you help me take a look? Note: Some test can not execute in http2 mode, so i disable it by --- no_http2 config command for skip http2 test.

oowl avatar Oct 24 '24 09:10 oowl

sister PR: https://github.com/openresty/test-nginx/pull/176

oowl avatar Oct 24 '24 09:10 oowl