Lax / traffic-accounting-nginx-module

Monitor the incoming and outgoing traffic metrics in realtime for NGINX
https://github.com/Lax/traffic-accounting-nginx-module
BSD 2-Clause "Simplified" License
280 stars 77 forks source link

Constant crashing #57

Open hpeshev opened 2 years ago

hpeshev commented 2 years ago

Hello, I had to compile the module for nginx version 1.17.7 and after i add the directive accounting on; it starts to start and kill workers:

`2022/01/12 13:25:25 [alert] 1#1: worker process 452 exited on signal 11
2022/01/12 13:25:25 [notice] 1#1: start worker process 453
2022/01/12 13:25:25 [notice] 1#1: signal 29 (SIGIO) received
2022/01/12 13:25:25 [notice] 1#1: signal 29 (SIGIO) received
2022/01/12 13:25:25 [notice] 1#1: signal 17 (SIGCHLD) received from 451
2022/01/12 13:25:25 [alert] 1#1: worker process 451 exited on signal 11
2022/01/12 13:25:25 [notice] 1#1: start worker process 454
....
`

i used the ./configuration from issue #42 changing the version to 17.7 but it still crushes. My nginx -V

`nginx version: nginx/1.17.7
built by gcc 9.3.0 (Alpine 9.3.0)
built with OpenSSL 1.1.1j  16 Feb 2021
TLS SNI support enabled
configure arguments: --prefix=/usr/local/nginx --sbin-path=/usr/local/nginx/sbin/nginx --conf-path=/usr/local/nginx/conf/nginx.conf --pid-path=/usr/local/nginx/logs/nginx.pid --error-log-path=/var/log/nginx/error.log --http-log-path=/var/log/nginx/access.log --with-ld-opt=-Wl,-rpath,/usr/lib --user=nginx --group=nginx --with-pcre-jit --add-module=/tmp/tmp.BcBLcl/nginx_upstream_check_module-0.3.0 --add-module=/tmp/tmp.BcBLcl/ngx_devel_kit-0.3.1 --add-module=/tmp/tmp.BcBLcl/lua-nginx-module-0.10.15 --add-module=/tmp/tmp.BcBLcl/spnego-http-auth-nginx-module-1.1.0 --with-http_ssl_module --with-stream_ssl_module --with-http_sub_module --with-stream --with-threads --with-http_realip_module --with-file-aio --with-http_auth_request_module --with-compat --with-http_stub_status_module --with-http_v2_module`

I also tryed to compile the module with these options but it starts crushing with load-module,

dvershinin commented 2 years ago

@hborislavos the issue is in the stream subsystem. if you modify the config like this, it will load fine later (of course, the accounting won't work within stream, but at least the HTTP one will work)