do not add includes for redirect vhosts

* adds a condition to only insert includes if a vhost file contains
  a root directive (to ignore vhosts which are simply redirects)

* fixes https://github.com/mitchellkrogza/nginx-ultimate-bad-bot-blocker/issues/138
This commit is contained in:
Stuart Cardall 2018-02-26 20:41:26 +00:00
parent f9e152b7f9
commit 21830c19b6
No known key found for this signature in database
GPG key ID: AEB857F1C891D0C6

View file

@ -390,10 +390,14 @@ main() {
# update vhosts
for file in $file_list; do
line=$(find_includes $file include last server_ last location first )
if [ -n "$(check_wildcard $file $BOTS_DIR)" ]; then
# do not use wildcards in vhost files
printf "%-10s %-${col_size}s %s\n" "WARN:" "$BOTS_DIR/* detected" "=> $file"
else # don't use wildcards in vhost files
add_includes $line $file $BOTS_DIR $col_size $VHOST_INCLUDES
else # do not add includes to vhosts without root directive (i.e redirects)
if grep -Ew [[:space:]]+root $file 1>/dev/null; then
add_includes $line $file $BOTS_DIR $col_size $VHOST_INCLUDES
fi
fi
done