-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
chore(patches): add patches from upstream openresty #12940
Conversation
### Summary #### ngx_lua patches - openresty/lua-nginx-module@e5248aa - openresty/lua-nginx-module@e2067dd - openresty/lua-nginx-module@6c00bd4 #### ngx_stream_lua patches - openresty/stream-lua-nginx-module@cb82db3 - openresty/stream-lua-nginx-module@bea8a0c Signed-off-by: Aapo Talvensaari <[email protected]>
@dndx / @chronolaw what do you think about this? |
It is very interesting, I will check them in the next workday. |
Successfully created cherry-pick PR for |
|
||
--- | ||
src/ngx_http_lua_util.c | 6 ++++ | ||
t/020-subrequest.t | 80 +++++++++++++++++++++++++++++++++++++++++ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the late comment. It seems the test modification is not included in our patches. Is it because we don't run any tests from openresty?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, we do not run any openresty test cases in our repo.
### Summary #### ngx_lua patches - openresty/lua-nginx-module@e5248aa - openresty/lua-nginx-module@e2067dd - openresty/lua-nginx-module@6c00bd4 #### ngx_stream_lua patches - openresty/stream-lua-nginx-module@cb82db3 - openresty/stream-lua-nginx-module@bea8a0c Signed-off-by: Aapo Talvensaari <[email protected]> (cherry picked from commit 7a1d8d2)
Summary
ngx_lua patches
ngx_stream_lua patches
KAG-4379