search

kyprizel / testcookie-nginx-module

simple robot mitigation module using cookie based challenge/response technique. Not supported any more.
http://kyprizel.github.com/testcookie-nginx-module/
524 stars 139 forks source link

Added support for nginx 1.23.x #77

Closed 0xfe0 closed 2 years ago

0xfe0 commented 2 years ago

This PR fixes https://github.com/kyprizel/testcookie-nginx-module/issues/76.

I'm maintaining compatibility with old versions (pre 1.23.x) too, but in my opinion it would be a good idea to remove these checks and leave support for nginx >= 1.23 only once nginx publishes a new stable release.

Also, not related to this PR but I think it would also be a nice to put all debug prints inside NGX_DEBUG checks.

What do you think about these two suggestions @kyprizel?

kyprizel commented 2 years ago

can you keep the existing formatting?

0xfe0 commented 2 years ago

can you keep the existing formatting?

Done.

kyprizel commented 2 years ago

Thank you!