perusio / nginx-spamhaus-drop

A shell script that creates a Nginx configuration for using Spamhaus DROP lists

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Incorrect configuration instrutcions in the README.md

supersophie opened this issue · comments

The config settigs suggested for nginx are incorrect. The file is downloaded to /etc/nginx/drop_list.conf but the READ.md tells the user to add nginx_spamhaus_drop_list.conf to the nginx.conf

# ./nginx-drop-fetch 
File drop.txt adding 822 networks
File edrop.txt adding 56 networks
File /etc/nginx/drop_list.conf written.

    ## Include the Spamhaus DROP lists.
        include nginx_spamhaus_drop_list.conf;

# nginx -t
nginx: [emerg] open() "/etc/nginx/nginx_spamhaus_drop_list.conf" failed (2: No such file or directory) in /etc/nginx/nginx.conf:82

On another note:
Although when I corrected the path it was sick anyway:
nginx: [emerg] invalid number of arguments in "include" directive in /etc/nginx/nginx.conf:89
nginx: configuration file /etc/nginx/nginx.conf test failed