nodogsplash2: init script cleanup and refactoring 395/head
authorMoritz Warning <moritzwarning@web.de>
Wed, 2 May 2018 19:35:39 +0000 (21:35 +0200)
committerMoritz Warning <moritzwarning@web.de>
Mon, 23 Jul 2018 20:25:15 +0000 (22:25 +0200)
- refactor firewall restart hook
- remove dead code
- shorten existing code

Signed-off-by: Moritz Warning <moritzwarning@web.de>
nodogsplash2/Makefile
nodogsplash2/files/etc/config/nodogsplash [new file with mode: 0644]
nodogsplash2/files/etc/init.d/nodogsplash [new file with mode: 0755]
nodogsplash2/files/etc/uci-defaults/40_nodogsplash [new file with mode: 0644]
nodogsplash2/files/nodogsplash.config [deleted file]
nodogsplash2/files/nodogsplash.init [deleted file]
nodogsplash2/files/usr/lib/nodogsplash/restart.sh [new file with mode: 0755]

index 56ffb8a6455930df6a8bd9a4cfb4e5a2e899ff7a..a306316434f947f57907fa66e7cae7516f4fcf19 100644 (file)
@@ -38,15 +38,12 @@ define Package/nodogsplash2/description
 endef
 
 define Package/nodogsplash2/install
+       $(CP) ./files/* $(1)/
+
        $(INSTALL_DIR) $(1)/usr/bin
        $(INSTALL_BIN) $(PKG_BUILD_DIR)/nodogsplash $(1)/usr/bin/
        $(INSTALL_BIN) $(PKG_BUILD_DIR)/ndsctl $(1)/usr/bin/
 
-       $(INSTALL_DIR) $(1)/etc/init.d
-       $(INSTALL_BIN) files/nodogsplash.init $(1)/etc/init.d/nodogsplash
-       $(INSTALL_DIR) $(1)/etc/config
-       $(INSTALL_CONF) files/nodogsplash.config $(1)/etc/config/nodogsplash
-
        $(INSTALL_DIR) $(1)/etc/nodogsplash/htdocs/images
        $(CP) $(PKG_BUILD_DIR)/resources/splash.html $(1)/etc/nodogsplash/htdocs/
        $(CP) $(PKG_BUILD_DIR)/resources/infoskel.html $(1)/etc/nodogsplash/htdocs/
diff --git a/nodogsplash2/files/etc/config/nodogsplash b/nodogsplash2/files/etc/config/nodogsplash
new file mode 100644 (file)
index 0000000..f32e93a
--- /dev/null
@@ -0,0 +1,75 @@
+
+# The options available here are an adaptation of the settings used in nodogsplash.conf.
+# See https://github.com/nodogsplash/nodogsplash/blob/master/resources/nodogsplash.conf
+
+config nodogsplash
+  # Set to 0 to disable nodogsplash
+  option enabled 1
+
+  # Set to 0 to disable hook that makes nodogsplash restart when the firewall restarts.
+  # This hook is needed as a restart of Firewall overwrites nodogsplash iptables entries.
+  option fwhook_enabled '1'
+
+  # Serve the file splash.html from this directory
+  option webroot '/etc/nodogsplash/htdocs'
+
+  # Use plain configuration file
+  #option config '/etc/nodogsplash/nodogsplash.conf'
+
+  # Use this option to set the device nogogsplash will bind to.
+  # The value may be an interface section in /etc/config/network or a device name such as br-lan.
+  option gatewayinterface 'br-lan'
+
+  option gatewayname 'OpenWrt Nodogsplash'
+  option maxclients '250'
+
+  # Enables debug output (0-7)
+  #option debuglevel '7'
+
+  # Client timeouts in minutes
+  option clientidletimeout '120'
+  option clientforcetimeout '1440'
+
+  # Your router may have several interfaces, and you
+  # probably want to keep them private from the network/gatewayinterface.
+  # If so, you should block the entire subnets on those interfaces, e.g.:
+  # list authenticated_users 'block to 192.168.0.0/16'
+  # list authenticated_users 'block to 10.0.0.0/8'
+
+  # Typical ports you will probably want to open up.
+  #list authenticated_users 'allow tcp port 22'
+  #list authenticated_users 'allow tcp port 53'
+  #list authenticated_users 'allow udp port 53'
+  #list authenticated_users 'allow tcp port 80'
+  #list authenticated_users 'allow tcp port 443'
+  # Or for happy customers allow all
+  list authenticated_users 'allow all'
+
+  # For preauthenticated users to resolve IP addresses in their
+  # initial request not using the router itself as a DNS server,
+  # Leave commented to help prevent DNS tunnelling
+  #list preauthenticated_users 'allow tcp port 53'
+  #list preauthenticated_users 'allow udp port 53'
+
+  # Allow ports for SSH/Telnet/DNS/DHCP/HTTP/HTTPS
+  list users_to_router 'allow tcp port 22'
+  list users_to_router 'allow tcp port 23'
+  list users_to_router 'allow tcp port 53'
+  list users_to_router 'allow udp port 53'
+  list users_to_router 'allow udp port 67'
+  list users_to_router 'allow tcp port 80'
+
+  # MAC addresses that are / are not allowed to access the splash page
+  # Value is either 'allow' or 'block'. The allowedmac or blockedmac list is used.
+  #option macmechanism 'allow'
+  #list allowedmac '00:00:C0:01:D0:0D'
+  #list allowedmac '00:00:C0:01:D0:1D'
+  #list blockedmac '00:00:C0:01:D0:2D'
+
+  # MAC addresses that do not need to authenticate
+  #list trustedmac '00:00:C0:01:D0:1D'
+
+  # Set FW_MARK for compatibilty with other OpenWrt Packages eg mwan3, sqm etc.
+  list fw_mark_authenticated '30000'
+  list fw_mark_trusted '20000'
+  list fw_mark_blocked '10000'
diff --git a/nodogsplash2/files/etc/init.d/nodogsplash b/nodogsplash2/files/etc/init.d/nodogsplash
new file mode 100755 (executable)
index 0000000..3834788
--- /dev/null
@@ -0,0 +1,205 @@
+#!/bin/sh /etc/rc.common
+
+#
+# Startup/shutdown script for nodogsplash captive portal
+#
+
+START=95
+STOP=95
+
+USE_PROCD=1
+
+IPT=/usr/sbin/iptables
+WD_DIR=/usr/bin
+# -s -d 5 runs in background, with level 5 (not so verbose) messages to syslog
+# -f -d 7 runs in foreground, with level 7 (verbose) debug messages to terminal
+OPTIONS="-s -f -d 5"
+CONFIG=""
+
+
+addline() {
+  append CONFIG "$1" "$N"
+}
+
+setup_mac_lists() {
+  local cfg="$1"
+  local macs=""
+  local val
+
+  append_mac() {
+    append macs "$1" ","
+  }
+
+  config_get val "$cfg" macmechanism
+  if [ -z "$val" ]; then
+    # Check if we have AllowedMACList or BlockedMACList defined they will be ignored
+    config_get val "$cfg" allowedmac
+    if [ -n "$val" ]; then
+      echo "Ignoring allowedmac - macmechanism not \"allow\"" >&2
+    fi
+
+    config_get val "$cfg" blockedmac
+    if [ -n "$val" ]; then
+      echo "Ignoring blockedmac - macmechanism not \"block\"" >&2
+    fi
+  elif [ "$val" = "allow" ]; then
+    config_list_foreach "$cfg" allowedmac append_mac
+    addline "AllowedMACList $macs"
+  elif [ "$val" = "block" ]; then
+    config_list_foreach "$cfg" blockedmac append_mac
+    addline "BlockedMACList $macs"
+  else
+    echo "Invalid macmechanism '$val' - allow or block are valid." >&2
+    exit 1
+  fi
+
+  macs=""
+  config_list_foreach "$cfg" trustedmac append_mac
+  if [ -n "$macs" ]; then
+    addline "TrustedMACList $macs"
+  fi
+}
+
+setup_firewall() {
+  local cfg="$1"
+  local uci_name
+  local val
+
+  append_firewall() {
+    addline "  FirewallRule $1"
+  }
+
+  for rule in authenticated-users preauthenticated-users users-to-router trusted-users trusted-users-to-router; do
+    # uci does not allow dashes
+    uci_name=${rule//-/_}
+    addline "FirewallRuleSet $rule {"
+    config_list_foreach "$cfg" "$uci_name" append_firewall
+    addline "}"
+    config_get val "$cfg" "policy_${uci_name}"
+    if [ -n "$val" ]; then
+      addline "EmptyRuleSetPolicy $rule $val"
+    fi
+  done
+}
+
+wait_for_interface() {
+  local ifname="$1"
+  local timeout=10
+
+  for i in $(seq $timeout); do
+    if [ $(ip -4 addr show dev $ifname 2> /dev/null | grep -c inet) -ne 0 ]; then
+      break
+    fi
+    sleep 1
+    if [ $i = $timeout ]; then
+      echo "Interface $ifname not detected." >&2
+      exit 1
+    fi
+  done
+}
+
+generate_uci_config() {
+  local cfg="$1"
+  local val
+  local ifname
+  local download
+  local upload
+
+  # Init config file content
+  CONFIG="# auto-generated config file from /etc/config/nodogsplash"
+
+  config_get val "$cfg" config
+  if [ -n "$val" ]; then
+    if [ ! -f "$val" ]; then
+      echo "Configuration file '$file' doesn't exist." >&2
+      exit 1
+    fi
+    addline "$(cat $val)"
+  fi
+
+  config_get ifname "$cfg" gatewayinterface
+  if [ -z "$ifname" ]; then
+    config_get ifname "$cfg" network
+  fi
+
+  # Get device name if interface name is a section name in /etc/config/network
+  if network_get_device tmp "$ifname"; then
+      ifname="$tmp"
+  fi
+
+  if [ -z "$ifname" ]; then
+      echo "Option network or gatewayinterface missing." >&2
+      exit 1
+  fi
+
+  wait_for_interface "$ifname"
+
+  addline "GatewayInterface $ifname"
+
+  for option in authenticateimmediately binvoucher checkinterval clientforcetimeout \
+    clientidletimeout debuglevel decongesthttpdthreads downloadlimit enablepreauth \
+    forcevoucher fw_mark_authenticated fw_mark_blocked fw_mark_trusted gatewayaddress \
+    gatewayiprange gatewayname gatewayport httpdthreaddelayms httpdthreadthreshold imagedir \
+    macmechanism maxclients pagesdir password passwordattempts passwordauthentication \
+    redirecturl remoteauthenticatoraction splashpage syslogfacility uploadlimit \
+    username usernameauthentication webroot
+  do
+    config_get val "$cfg" "$option"
+
+    if [ -n "$val" ]; then
+      addline "$option $val"
+    fi
+  done
+
+  config_get download "$cfg" downloadlimit
+  config_get upload "$cfg" uploadlimit
+
+  if [ -n "$upload" -o -n "$download" ]; then
+    addline "TrafficControl yes"
+  fi
+
+  setup_mac_lists "$cfg"
+  setup_firewall "$cfg"
+
+  echo "$CONFIG" > "/tmp/etc/nodogsplash_$cfg.conf"
+}
+
+# setup configuration and start instance
+create_instance() {
+  local cfg="$1"
+  local val
+
+  config_get_bool val "$cfg" enabled 0
+  [ $val -gt 0 ] || return 0
+
+  generate_uci_config "$cfg"
+
+  if ! test_module; then
+    echo "Nodogsplash is missing some kernel modules." >&2
+    exit 1
+  fi
+
+  procd_open_instance $cfg
+  procd_set_param command /usr/bin/nodogsplash -c "/tmp/etc/nodogsplash_$cfg.conf" $OPTIONS
+  procd_set_param respawn
+  procd_set_param file "/tmp/etc/nodogsplash_$cfg.conf"
+  procd_close_instance
+}
+
+start_service() {
+  # For network_get_device()
+  include /lib/functions
+
+  # For nodogsplash.conf file
+  mkdir -p /tmp/etc/
+
+  config_load nodogsplash
+  config_foreach create_instance nodogsplash
+}
+
+stop_service() {
+  # When procd terminates nodogsplash, it does not exit fast enough.
+  # Otherwise procd will restart nodogsplash twice. First time starting
+  # nodogsplash fails, second time it succeeds.
+  sleep 1
+}
diff --git a/nodogsplash2/files/etc/uci-defaults/40_nodogsplash b/nodogsplash2/files/etc/uci-defaults/40_nodogsplash
new file mode 100644 (file)
index 0000000..42181e5
--- /dev/null
@@ -0,0 +1,9 @@
+#!/bin/sh
+
+uci batch <<-EOT
+       delete firewall.nodogsplash2
+       set firewall.nodogsplash2=include
+       set firewall.nodogsplash2.type=script
+       set firewall.nodogsplash2.path=/usr/lib/nodogsplash/restart.sh
+       commit firewall
+EOF
diff --git a/nodogsplash2/files/nodogsplash.config b/nodogsplash2/files/nodogsplash.config
deleted file mode 100644 (file)
index 2107b77..0000000
+++ /dev/null
@@ -1,82 +0,0 @@
-
-# The options available here are an adaptation of the settings used in nodogsplash.conf.
-# See https://github.com/nodogsplash/nodogsplash/blob/master/resources/nodogsplash.conf
-
-config nodogsplash
-  # Set to 0 to disable nodogsplash
-  option enabled 1
-  
-  # Set to 0 to disable hook that makes Firewall restart nodogsplash when Firewall restarts
-  # This hook is needed as a restart of Firewall overwrites nodogsplash iptables entries
-  option fwhook_enabled '1'
-
-  # Serve the file splash.html from this directory
-  option webroot '/etc/nodogsplash/htdocs'
-
-  # Use plain configuration file
-  #option config '/etc/nodogsplash/nodogsplash.conf'
-
-  # Use this option to set the network interface the users are connected to
-  # Must not be used with option gatewayinterface
-  # This option automatically identifies the active lan device for nodogsplash to bind to
-  # This option may fail if the device configured for this interface is not up when nodogsplash starts at boot time
-  # You may change this to any valid virtual lan interface that has been defined, eg lan, lan2, public_lan wlan2 etc
-  # option network 'lan'
-  
-  # Use this option to set the device nogogsplash will bind to
-  # Must not be used with option network
-  # The nodogsplash init script will wait for this device to be up before loading the nodogsplash service
-  # You may change this to any valid lan device eg br-lan, wlan0, eth0.1 etc
-  option gatewayinterface 'br-lan'
-  
-  option gatewayname 'OpenWrt Nodogsplash'
-  option maxclients '250'
-  # Client timeouts in minutes
-  option clientidletimeout '120'
-  option clientforcetimeout '1440'
-
-
-  # Your router may have several interfaces, and you
-  # probably want to keep them private from the network/gatewayinterface.
-  # If so, you should block the entire subnets on those interfaces, e.g.:
-  # list authenticated_users 'block to 192.168.0.0/16'
-  # list authenticated_users 'block to 10.0.0.0/8'
-
-  # Typical ports you will probably want to open up.
-  #list authenticated_users 'allow tcp port 22'
-  #list authenticated_users 'allow tcp port 53'
-  #list authenticated_users 'allow udp port 53'
-  #list authenticated_users 'allow tcp port 80'
-  #list authenticated_users 'allow tcp port 443'
-  # Or for happy customers allow all
-  list authenticated_users 'allow all'
-
-
-  # For preauthenticated users to resolve IP addresses in their
-  # initial request not using the router itself as a DNS server,
-  # Leave commented to help prevent DNS tunnelling
-  #list preauthenticated_users 'allow tcp port 53'
-  #list preauthenticated_users 'allow udp port 53'
-
-  # Allow ports for SSH/Telnet/DNS/DHCP/HTTP/HTTPS
-  list users_to_router 'allow tcp port 22'
-  list users_to_router 'allow tcp port 23'
-  list users_to_router 'allow tcp port 53'
-  list users_to_router 'allow udp port 53'
-  list users_to_router 'allow udp port 67'
-  list users_to_router 'allow tcp port 80'
-
-  # MAC addresses that are / are not allowed to access the splash page
-  # Value is either 'allow' or 'block'. The allowedmac or blockedmac list is used.
-  #option macmechanism 'allow'
-  #list allowedmac '00:00:C0:01:D0:0D'
-  #list allowedmac '00:00:C0:01:D0:1D'
-  #list blockedmac '00:00:C0:01:D0:2D'
-
-  # MAC addresses that do not need to authenticate
-  #list trustedmac '00:00:C0:01:D0:1D'
-
-  # Set FW_MARK for compatibilty with other OpenWrt Packages eg mwan3, sqm etc.
-  list fw_mark_authenticated '30000'
-  list fw_mark_trusted '20000'
-  list fw_mark_blocked '10000'
diff --git a/nodogsplash2/files/nodogsplash.init b/nodogsplash2/files/nodogsplash.init
deleted file mode 100755 (executable)
index 87229c9..0000000
+++ /dev/null
@@ -1,387 +0,0 @@
-#!/bin/sh /etc/rc.common
-#
-# description: Startup/shutdown script for nodogsplash captive portal
-#
-# Alexander Couzens <lynxis@fe80.eu> 2014
-# P. Kube 2007
-#
-# (Based on wifidog startup script
-# Date    : 2004-08-25
-# Version : 1.0
-# Comment by that author: Could be better, but it's working as expected)
-#
-
-START=95
-STOP=95
-
-USE_PROCD=1
-
-IPT=/usr/sbin/iptables
-WD_DIR=/usr/bin
-# -s -d 5 runs in background, with level 5 (not so verbose) messages to syslog
-# -f -d 7 runs in foreground, with level 7 (verbose) debug messages to terminal
-OPTIONS="-s -f -d 5"
-CONFIGFILE="/tmp/invalid_nodogsplash.conf"
-
-# nolog(loglevel message ...)
-nolog() {
-  local level=$1
-  shift
-  logger -s -t nodogsplash -p daemon.$level $@
-}
-
-# append_config_option_map <cfgfile> <uci_cfg_obj> <option_name> <config_counterpart> [<optional default>]
-# append "$config_counterpart $value" to cfgfile if option_name exists
-# e.g. append_config_option "$CONFIGFILE" "$cfg" bind_address BindAddress 0.0.0.0
-# will append "BindAddress 192.168.1.1" if uci bind_address is '192.168.1.1'
-append_config_option_map() {
-  local val=""
-  local config_file="$1"
-  local cfg="$2"
-  local option_name="$3"
-  local config_counterpart="$4"
-  local default="$5"
-  config_get val "$cfg" "$option_name" "$default"
-  [ -n "$val" ] && echo "$config_counterpart $val" >> $config_file
-}
-
-# append_config_option <cfgfile> <uci_cfg_obj> <option_name> [<optional default>]
-# append "$option_name $value" to cfgfile if option_name exists
-# e.g. append_config_option "$CONFIGFILE" "$cfg" bind_address 0.0.0.0
-# will append "bind_address 192.168.1.1" if uci bind_address is '192.168.1.1'
-# if uci bind_address is unset append "bind_address 0.0.0.0"
-append_config_option() {
-  local val=""
-  local config_file="$1"
-  local cfg="$2"
-  local option_name="$3"
-  local default="$4"
-  config_get val "$cfg" "$option_name" "$default"
-  [ -n "$val" ] && echo "$option_name $val" >> $config_file
-}
-
-setup_mac_lists() {
-  local cfg="$1"
-  local MAC=""
-  local val
-
-  append_mac() {
-    append MAC "$1" ","
-  }
-
-  config_get val "$cfg" macmechanism
-  if [ -z "$val" ] ; then
-    # check if we have AllowedMACList or BlockedMACList defined they will be ignored
-    config_get val "$cfg" allowedmac
-    if [ -n "$val" ] ; then
-      echo "Ignoring allowedmac - macmechanism not \"allow\"" >&2
-    fi
-
-    config_get val "$cfg" blockedmac
-    if [ -n "$val" ] ; then
-      echo "Ignoring blockedmac - macmechanism not \"block\"" >&2
-    fi
-  elif [ "$val" == "allow" ] ; then
-    MAC=""
-    config_list_foreach "$cfg" allowedmac append_mac
-    echo "AllowedMACList $MAC" >> $CONFIGFILE
-  elif [ "$val" == "block" ] ; then
-    MAC=""
-    config_list_foreach "$cfg" blockedmac append_mac
-    echo "BlockedMACList $MAC" >> $CONFIGFILE
-  else
-    nolog error "$cfg Invalid macmechanism '$val' - allow or block are valid."
-    return 1
-  fi
-  MAC=""
-  config_list_foreach "$cfg" trustedmac append_mac
-  [ -n "$MAC" ] && echo "TrustedMACList $MAC" >> $CONFIGFILE
-}
-
-setup_firewall() {
-  local cfg="$1"
-  local uciname
-  local val
-
-  append_firewall() {
-    echo "    FirewallRule $1" >> $CONFIGFILE
-  }
-
-  for rule in $(echo authenticated-users preauthenticated-users users-to-router trusted-users trusted-users-to-router)
-  do
-    uci_name=${rule//-/_}
-    # uci does not allow - dashes
-    echo "FirewallRuleSet $rule {" >> $CONFIGFILE
-    config_list_foreach "$cfg" ${uci_name} append_firewall
-    echo "}" >> $CONFIGFILE
-    config_get val "$cfg" policy_${uci_name}
-    [ -n "$val" ] && echo "EmptyRuleSetPolicy $rule $val" >> $CONFIGFILE
-  done
-}
-
-wait_for_interface()
-{
-  local ifname="$1"
-  local timeout=10
-  for i in $(seq $timeout); do
-    if [ $(ip -4 addr show dev $ifname 2> /dev/null | grep -c inet) -ne 0 ]; then
-      break
-    fi
-    sleep 1
-    if [ $i == $timeout ] ; then
-      nolog error "$ifname not detected, NoDogSplash not starting."
-      exit 1
-    fi
-  done
-}
-
-generate_uci_config() {
-  local cfg="$1"
-  local val
-  local ifname
-  local download
-  local upload
-
-  CONFIGFILE="/tmp/etc/nodogsplash_$cfg.conf"
-
-  echo "# auto-generated config file from /etc/config/nodogsplash" > $CONFIGFILE
-
-  config_get val "$cfg" config
-  if [ -n "$val" ] ; then
-    if [ ! -f "$val" ] ; then
-      nolog error "Configuration file '$file' doesn't exist"
-      return 0
-    fi
-    cat "$val" >> $CONFIGFILE
-  fi
-
-  config_get val "$cfg" network
-  if [ -n "$val" ] ; then
-    if ! network_get_device ifname "$val" ; then
-      nolog error "$cfg can not find ifname for network '$val'"
-      return 1
-    fi
-  fi
-
-  config_get val "$cfg" gatewayinterface
-  if [ -n "$val" ] ; then
-    if [ -n "$ifname" ] ; then
-      nolog error "$cfg cannot use both option network and gatewayinterface"
-      return 1
-    fi
-    ifname="$val"
-  fi
-
-  if [ -z "$ifname" ] ; then
-      nolog error "$cfg option network or gatewayinterface missing"
-      return 1
-  fi
-
-  wait_for_interface "$ifname"
-
-  echo "GatewayInterface $ifname" >> $CONFIGFILE
-
-  append_config_option "$CONFIGFILE" "$cfg" gatewayname
-  append_config_option "$CONFIGFILE" "$cfg" gatewayaddress
-  append_config_option "$CONFIGFILE" "$cfg" gatewayport
-  append_config_option "$CONFIGFILE" "$cfg" maxclients
-  append_config_option "$CONFIGFILE" "$cfg" webroot
-  append_config_option "$CONFIGFILE" "$cfg" debuglevel
-  append_config_option "$CONFIGFILE" "$cfg" splashpage
-  append_config_option "$CONFIGFILE" "$cfg" pagesdir
-  append_config_option "$CONFIGFILE" "$cfg" checkinterval
-  append_config_option "$CONFIGFILE" "$cfg" syslogfacility
-  append_config_option "$CONFIGFILE" "$cfg" gatewayiprange
-  append_config_option "$CONFIGFILE" "$cfg" imagedir
-  append_config_option "$CONFIGFILE" "$cfg" redirecturl
-  append_config_option "$CONFIGFILE" "$cfg" clientidletimeout
-  append_config_option "$CONFIGFILE" "$cfg" clientforcetimeout
-  append_config_option "$CONFIGFILE" "$cfg" gatewayiprange
-  append_config_option "$CONFIGFILE" "$cfg" passwordattempts
-  append_config_option "$CONFIGFILE" "$cfg" macmechanism
-  append_config_option "$CONFIGFILE" "$cfg" uploadlimit
-  append_config_option "$CONFIGFILE" "$cfg" downloadlimit
-  append_config_option "$CONFIGFILE" "$cfg" remoteauthenticatoraction
-  append_config_option "$CONFIGFILE" "$cfg" enablepreauth
-  append_config_option "$CONFIGFILE" "$cfg" binvoucher
-  append_config_option "$CONFIGFILE" "$cfg" forcevoucher
-  append_config_option "$CONFIGFILE" "$cfg" passwordauthentication
-  append_config_option "$CONFIGFILE" "$cfg" usernameauthentication
-  append_config_option "$CONFIGFILE" "$cfg" passwordattempts
-  append_config_option "$CONFIGFILE" "$cfg" username
-  append_config_option "$CONFIGFILE" "$cfg" password
-  append_config_option "$CONFIGFILE" "$cfg" authenticateimmediately
-  append_config_option "$CONFIGFILE" "$cfg" decongesthttpdthreads
-  append_config_option "$CONFIGFILE" "$cfg" httpdthreadthreshold
-  append_config_option "$CONFIGFILE" "$cfg" httpdthreaddelayms
-  append_config_option "$CONFIGFILE" "$cfg" fw_mark_authenticated
-  append_config_option "$CONFIGFILE" "$cfg" fw_mark_trusted
-  append_config_option "$CONFIGFILE" "$cfg" fw_mark_blocked
-
-  config_get download "$cfg" downloadlimit
-  config_get upload "$cfg" uploadlimit
-  [ -n "$upload" -o -n "$download" ] && echo "TrafficControl yes" >> $CONFIGFILE
-
-  setup_mac_lists "$cfg"
-  setup_firewall "$cfg"
-}
-
-# setup configuration and start instance
-create_instance() {
-  local cfg="$1"
-  local manual_config
-  local val
-
-  config_get_bool val "$cfg" enabled 0
-  [ $val -gt 0 ] || return 0
-
-  generate_uci_config "$cfg"
-
-  if ! test_module ;  then
-    logger -s -t nodogsplash -p daemon.error "nodogsplash is missing some kernel modules"
-  fi
-
-  procd_open_instance $cfg
-  procd_set_param command /usr/bin/nodogsplash -c $CONFIGFILE $OPTIONS
-  procd_set_param respawn
-  procd_set_param file $CONFIGFILE
-  procd_close_instance
-}
-
-depends() {
-  if [ "$1" = "iptables" ] ; then
-    if [ $(uci get nodogsplash.@nodogsplash[0].fwhook_enabled) = "1" ] ; then
-      if $WD_DIR/ndsctl status > /dev/null; then
-       echo " * Restarting NodogSplash" 
-       /etc/init.d/nodogsplash restart
-      fi
-    else
-      echo " * NodogSplash fwhook is disabled"
-    fi
-  fi
-}
-
-start_service() {
-  include /lib/functions
-  mkdir -p /tmp/etc/
-  
-  if [ $(uci get nodogsplash.@nodogsplash[0].fwhook_enabled) = "1" ] ; then
-    if ! uci get firewall.nodogsplash2.path &> /dev/null ; then
-      if [ -f '/tmp/etc/ndshook.include' ] ; then
-        rm /tmp/etc/ndshook.include
-      fi
-      uci delete firewall.nodogsplash2 2> /dev/null
-      uci set firewall.nodogsplash2=include
-      uci set firewall.nodogsplash2.type=script
-      uci set firewall.nodogsplash2.path='/tmp/etc/ndshook.include'
-      uci commit firewall
-      /etc/init.d/firewall restart  2>&1 | logger
-    fi
-
-    if [ ! -f '/tmp/etc/ndshook.include' ] ; then
-      printf "if [ -f '/etc/init.d/nodogsplash' ] ; then /etc/init.d/nodogsplash depends iptables ; fi\n" > /tmp/etc/ndshook.include
-      chmod +x /tmp/etc/ndshook.include
-    fi
-  fi
-  
-  config_load nodogsplash
-
-  config_foreach create_instance nodogsplash
-}
-
-stop_service() {
-  # nodogsplash doesn't exit fast enought, when procd terminates it.
-  # otherwise procd will restart nodogsplash twice. first time starting nodogsplash fails, second time it succeeds
-  sleep 1
-}
-
-status() {
-  $WD_DIR/ndsctl status
-}
-
-# Test if we got all modules loaded
-test_module() {
-  ### Test ipt_mark with iptables
-  test_ipt_mark () {
-    ($IPT -A FORWARD -m mark --mark 2 -j ACCEPT 2>&1) > /dev/null
-    IPTABLES_OK=$?
-    if [ "$IPTABLES_OK" -eq 0 ]; then
-      ($IPT -D FORWARD -m mark --mark 2 -j ACCEPT 2>&1) > /dev/null
-      return 0
-    else
-      return 1
-    fi
-  }
-
-  ### Test ipt_mac with iptables
-  test_ipt_mac () {
-    ($IPT -A INPUT -m mac --mac-source 00:00:00:00:00:00 -j ACCEPT 2>&1) > /dev/null
-    IPTABLES_OK=$?
-    if [ "$IPTABLES_OK" -eq 0 ]; then
-      ($IPT -D INPUT -m mac --mac-source 00:00:00:00:00:00 -j ACCEPT 2>&1) > /dev/null
-      return 0
-    else
-      return 1
-    fi
-  }
-
-  ### Test ipt_IMQ with iptables
-  test_ipt_IMQ () {
-    ($IPT -t mangle -A PREROUTING -j IMQ --todev 0 2>&1) > /dev/null
-    IPTABLES_OK=$?
-    if [ "$IPTABLES_OK" -eq 0 ]; then
-      ($IPT -t mangle -D PREROUTING -j IMQ --todev 0 2>&1) > /dev/null
-      return 0
-    else
-      return 1
-    fi
-  }
-
-  ### Test imq with ip
-  test_imq () {
-    (ip link set imq0 up 2>&1) > /dev/null
-    IMQ0_OK=$?
-    (ip link set imq1 up 2>&1) > /dev/null
-    IMQ1_OK=$?
-    if [ "$IMQ0_OK" -eq 0 -a "$IMQ1_OK" -eq 0 ]; then
-      (ip link set imq0 down 2>&1) > /dev/null
-      (ip link set imq1 down 2>&1) > /dev/null
-      return 0
-    else
-      return 1
-    fi
-  }
-
-  ### Test sch_htb with tc; requires imq0
-  test_sch_htb () {
-    (tc qdisc del dev imq0 root 2>&1) > /dev/null
-    (tc qdisc add dev imq0 root htb 2>&1) > /dev/null
-    TC_OK=$?
-    if [ "$TC_OK" -eq 0 ]; then
-      (tc qdisc del dev imq0 root 2>&1) > /dev/null
-      return 0
-    else
-      return 1
-    fi
-  }
-
-  ### Find a module on disk
-  module_exists () {
-    EXIST=$(find /lib/modules/`uname -r` -name $1.*o 2> /dev/null)
-    if [ -n "$EXIST" ]; then
-      return 0
-    else
-      return 1
-    fi
-  }
-
-  ### Test if a module is in memory
-  module_in_memory () {
-    MODULE=$(lsmod | grep $1 | awk '{print $1}')
-    if [ "$MODULE" = "$1" ]; then
-      return 0
-    else
-      return 1
-    fi
-  }
-}
diff --git a/nodogsplash2/files/usr/lib/nodogsplash/restart.sh b/nodogsplash2/files/usr/lib/nodogsplash/restart.sh
new file mode 100755 (executable)
index 0000000..e67f11d
--- /dev/null
@@ -0,0 +1,8 @@
+#!/bin/sh
+
+# Check if nodogsplash is running
+if ndsctl status &> /dev/null; then
+  if [ "$(uci -q get nodogsplash.@nodogsplash[0].fwhook_enabled)" = "1" ]; then
+    /etc/init.d/nodogsplash restart
+  fi
+fi