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

Published 20 hours ago • verified publisher icon openresty

Downstream UDP support

Open alonbg opened this issue 9 years ago • 7 comments

Original pull-request was closed as I branched this feature during some repo house keeping.

alonbg avatar Sep 08 '16 12:09 alonbg

&ngx_stream_lua_skcoet_tcp_metatable_key

skcoet -> socket

bjne avatar Oct 25 '16 20:10 bjne

@agentzh, t/099-c-api.t passes on my end but fails on the CI (unable to connect) Any ideas? Thanks

alonbg avatar Nov 01 '16 16:11 alonbg

@alonbg Seems like 099-c-api.t causes the nginx server process to crash. Better use valgrind to locate the memory issues on your side.

agentzh avatar Nov 01 '16 16:11 agentzh

export TEST_NGINX_USE_VALGRIND=1; ./go prove -r t/099-c-api.t 1> out.txt 2>&1 @agentzh, sorry :( but I didn't know what to make out of it. attached out.txt

alonbg avatar Nov 01 '16 17:11 alonbg

@agentzh @alonbg I think the 099-c-api.t failed is because the ngx_http_lua_module changed. I have created a PR for that https://github.com/openresty/stream-lua-nginx-module/pull/44

And, @agentzh have mark these test cases SKIP, so rebase the lastest master will goes right :)

doujiang24 avatar Nov 15 '16 10:11 doujiang24

@alonbg I think we'd better create a new directory like t/087-udp-socket and move the udp test files into it. We'd better keep the same test file number as in lua-nginx-module, what do you think?

doujiang24 avatar Nov 19 '16 06:11 doujiang24

When would this PR be merged/released? badly needed:(

monkeylite avatar Mar 09 '17 18:03 monkeylite