tokers / zstd-nginx-module

Nginx modules for the Zstandard compression
BSD 2-Clause "Simplified" License
196 stars 23 forks source link

add support for dynamic modules #10

Closed eilandert closed 2 years ago

eilandert commented 2 years ago

This PR adds support for dynamic modules, I am testing it now on my webserver on https://deb.myguard.nl (feel free to query or test the http responses). There are ubuntu/debian packages on http://edge.deb.myguard.nl:8888/pool/main/n/nginx/ for testing.

Source is splitted from src/ to static/ and filter/ to make 2 modules. removed the dir src/

$ ./configure --add-dynamic-module=debian/modules/http-zstd checking for ZStandard static library ... found checking for ZStandard static library ... found + ngx_zstd was configured

$ make -j100 blablabla make[1]: Leaving directory '/opt/packages/nginx/tmp/nginx-1.21.5' $ ls objs/*.so objs/ngx_http_zstd_filter_module.so objs/ngx_http_zstd_static_module.so

$ curl -s -I -H 'Accept-Encoding: zstd' https://deb.myguard.nl | grep zstd content-encoding: zstd

I might have trashed too much, please compare/test if you need to have it static