Vasfed / nginx_ipset_blacklist

nginx module to use linux netfilter ipsets as blacklists
43 stars 18 forks source link

Incompatibility with ipset 6 #4

Open MindHunter86 opened 8 years ago

MindHunter86 commented 8 years ago

Hi! We already have new version of IPSET from netfilter.

Can you upgrade this module? It's very useful fro nginx but is not compatible now =(

MindHunter86 commented 8 years ago

А если по-русски. Модуль ахерителен, но он сейчас вообще не совместим с новым модулем ipset. Если ли какая-нить вероятность того, что модуль проагрейдят ?

Vasfed commented 8 years ago

I'm not currently using this in production, so it will take some time to look into. Can you please provide more info on how it is incompatible?

Спасибо) Вероятность есть, но будет скорее всего небыстро. А какого вида там несовместимость - оно не компилится/падает?

MindHunter86 commented 8 years ago

Предлагаю по-русски только: Да блин. У IPSET поменялась апиха. Версия там уже:

ipset --version
ipset v6.19, protocol version: 6

Следовательно все header файлы тоже. Первое руганье на версию. Если вырубить в коде проверку, то после падает со сегфолтом.

А вообще модуль реально афигителен и очень юзабелен в проде. К сожалению, я так и не понял принцип программирования под него, да и вообще с C не дружу особо. Я все же админ)

methuselah-0 commented 6 years ago

in the nginx source tree: nginx-1.12.1/objs/Makefile you can remove the -Werror flag as a workaround. It's only a warning and doesn't seem very important in this case. Then you can compile as normal.