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

fix table concat bug #70

Open
wants to merge 5 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@zheng-ji

zheng-ji commented Oct 27, 2015

fix /ngx_lua_waf/waf.lua:75: invalid value (boolean) at index 1 in table for 'concat'
related Issue:#69

fix table concat bug
fix /ngx_lua_waf/waf.lua:75: invalid value (boolean) at index 1 in table for 'concat' 
Issue:#69
@loveshell

This comment has been minimized.

Show comment
Hide comment
@loveshell

loveshell Oct 28, 2015

fix table concat bug

loveshell commented on c7ce9b7 Oct 28, 2015

fix table concat bug

@zheng-ji

This comment has been minimized.

Show comment
Hide comment
@zheng-ji

zheng-ji Nov 8, 2015

Hi LoveShell:

我们在实际应用中,会有这样的场景:

在同一个 nginx 软件下,有一些server_name 出于某种客观原因,不希望使用waf,
于是我给ngx_lua_waf 添加 server_name 白名单,给指定的 server_name 放行.提了一个Pull Request 给你:)

再次感谢你的开源,希望可以一起努力让这个世界更好。

zheng-ji commented Nov 8, 2015

Hi LoveShell:

我们在实际应用中,会有这样的场景:

在同一个 nginx 软件下,有一些server_name 出于某种客观原因,不希望使用waf,
于是我给ngx_lua_waf 添加 server_name 白名单,给指定的 server_name 放行.提了一个Pull Request 给你:)

再次感谢你的开源,希望可以一起努力让这个世界更好。
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment