Skip to content
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
Choose a base branch
from
Open

fix table concat bug #70

wants to merge 5 commits into from

Conversation

zheng-ji
Copy link

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

@zheng-ji
Copy link
Author

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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants