Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fixed fd leackage after reload #139

Merged
merged 1 commit into from Sep 4, 2013

Conversation

Projects
None yet
2 participants
Contributor

chaizhenhua commented Aug 27, 2013

No description provided.

Contributor

brenosilva commented Sep 3, 2013

Could you just confirm is the patch doesn't bring back the 'nginx reload' issue ? Not sure if you already tested it.

Thanks!

Contributor

chaizhenhua commented Sep 4, 2013

hello, brenosilva.
I try reload several time, with following config. NGINX did not crash.

SecRuleEngine On
SecRule ARGS "@rx attack" "phase:1,log,deny,id:1"

@brenosilva brenosilva added a commit that referenced this pull request Sep 4, 2013

@brenosilva brenosilva Merge pull request #139 from chaizhenhua/remotes/trunk
Fixed fd leackage after reload
10fd40f

@brenosilva brenosilva merged commit 10fd40f into SpiderLabs:remotes/trunk Sep 4, 2013

Contributor

brenosilva commented Sep 4, 2013

Thanks!

@zimmerle zimmerle added a commit that referenced this pull request Oct 18, 2013

@zimmerle zimmerle Revert "Merge pull request #139 from chaizhenhua/remotes/trunk"
This reverts commit 10fd40f, reversing
changes made to 414033a.
6bda719

@zimmerle zimmerle added a commit to zimmerle/ModSecurity that referenced this pull request Oct 18, 2013

@zimmerle zimmerle Revert "Merge pull request #139 from chaizhenhua/remotes/trunk"
This reverts commit 10fd40f, reversing
changes made to 414033a.
7f7d00f

@zimmerle zimmerle added a commit that referenced this pull request Oct 18, 2013

@zimmerle zimmerle Merge pull request #579 from zimmerle/revert_139
Revert merge request #139
61e54f2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment