fix dash shell compatibilities install / setup-ngxblocker

fixes https://github.com/mitchellkrogza/nginx-ultimate-bad-bot-blocker/issues/39

also fixes a small spacing bug in setup-ngxblocker => nginx.conf
This commit is contained in:
Stuart Cardall 2017-04-27 19:22:51 +00:00
parent cce45d487f
commit 4afe3094ea
No known key found for this signature in database
GPG key ID: EB601090DB74C297
2 changed files with 12 additions and 10 deletions

View file

@ -129,9 +129,9 @@ download_files() {
}
check_config() {
local x=
local x= dirs="$*"
for x in "$@"; do
for x in $dirs; do
if [ ! -d $x ]; then
printf "Creating directory: $x\n"
if [ "$DRY_RUN" = "N" ]; then
@ -268,3 +268,4 @@ exit $?
# PLEASE ALSO SEE THE SETUP SCRIPT TO INSERT THE NECESSARY INCLUDES FOR YOU
### You can now use a setup script contributed by Stuart Cardall to automatically add the includes for you
### See - https://raw.githubusercontent.com/mitchellkrogza/nginx-ultimate-bad-bot-blocker/master/setup-ngxblocker

View file

@ -48,7 +48,7 @@ EOF
}
check_config() {
local files=$@
local files="$*"
if [ -z "$files" ]; then
echo "no vhost files in: $VHOST_DIR/*.$VHOST_EXT => exiting."
@ -111,7 +111,7 @@ longest_str() {
add_includes() {
local ph='<<!!>>' line=$1 file=$2 conf_dir=$3 text= update=
local include_list=$(echo $@ | awk '{$1=$2=$3=""}sub("^"OFS"+","")')
local include_list="$(echo $@ | awk '{$1=$2=$3=""}sub("^"OFS"+","")')"
local col_size=$(( $(longest_str $include_list) + $(echo $conf_dir | wc -m) ))
for text in $include_list; do
@ -175,10 +175,10 @@ find_includes() {
last) line=$(( $line + $start +1 ));;
esac
# if inserting beyond the end of the file
if [ $line -gt $end ]; then
# if inserting beyond the end of the stanza
if [ "$(sed -n $(( $line - 1))p $file | grep ^})" = "}" ]; then
# insert blank line
sed -i "$end i \ " $file
sed -i "$(( line - 1)) i \ " $file
fi
echo $line
@ -252,7 +252,7 @@ wget_opts() {
check_online() {
local url=$1 options=$(wget_opts)
if wget $options $url &>/dev/null; then
if wget $options $url 2>&1 /dev/null; then
echo "true"
fi
}
@ -277,7 +277,8 @@ main() {
if [ -n "$(check_online $include_url)" ]; then
local tmp=$(mktemp)
wget -q $include_url -O $tmp
source $tmp 2>/dev/null
# use period not source in POSIX shell
. $tmp 2>/dev/null
rm -f $tmp
else
printf "Repo down or missing: $include_url\n"
@ -286,7 +287,7 @@ main() {
# double check we have some files sourced
if [ -z "$CONF_FILES" ] || [ -z "$BOT_FILES" ]; then
printf "Error sourcing variables from: $include_url"
printf "Error sourcing variables from: $include_url\n"
exit 1
fi