]> git.ipfire.org Git - people/ms/network.git/commitdiff
hooks: Import zone default settings, too
authorMichael Tremer <michael.tremer@ipfire.org>
Sat, 30 Mar 2019 12:10:58 +0000 (13:10 +0100)
committerMichael Tremer <michael.tremer@ipfire.org>
Sat, 30 Mar 2019 12:10:58 +0000 (13:10 +0100)
Signed-off-by: Michael Tremer <michael.tremer@ipfire.org>
src/header-zone
src/hooks/zones/bridge
src/hooks/zones/ip-tunnel
src/hooks/zones/modem
src/hooks/zones/pppoe

index ead4a32c0512fb6170f6191ead4544286cd217e6..7ad3e39e5e7bf1033e8439e61f6bb28c3354acc6 100644 (file)
@@ -34,6 +34,11 @@ hook_new() {
        assert isset zone
        shift
 
+       local ${HOOK_SETTINGS}
+
+       # Import all default variables
+       hook_set_defaults
+
        if ! hook_parse_cmdline "$@"; then
                return ${EXIT_ERROR}
        fi
index 93a3a314d59a89a636380a631779e68549301a5d..33d5811270623a9c9825fa9fac581d4764882b73 100644 (file)
@@ -29,6 +29,7 @@ HOOK_SETTINGS="${HOOK_SETTINGS} STP_PRIORITY MTU"
 HOOK_PORT_SETTINGS="COST PRIORITY"
 
 # Default values
+DEFAULT_STP="on"
 DEFAULT_STP_FORWARD_DELAY=0
 DEFAULT_STP_HELLO=2
 DEFAULT_STP_MAXAGE=20
@@ -123,19 +124,11 @@ hook_parse_cmdline() {
                shift
        done
 
-       # Generate a random MAC address if the user passed no one
+       # Generate a random MAC address if the user passed none
        if ! isset ADDRESS; then
                ADDRESS="$(mac_generate)"
        fi
 
-       # Enable Spanning Tree Protocol by default
-       if ! isset STP; then
-               STP="on"
-       fi
-
-       # Set all other defaults
-       hook_set_defaults
-
        return ${EXIT_OK}
 }
 
index c9c73bab7472845538d16d66acee9199bf007a14..e4be361a97f4b91e2d08351aeed1628e66573667 100644 (file)
@@ -26,13 +26,7 @@ SUPPORTED_IP_TUNNEL_MODES="gre vti"
 HOOK_SETTINGS="HOOK MARK MODE PEER LOCAL_ADDRESS"
 
 # Default mode of the tunnel
-MODE="gre"
-
-# The IP address of the tunnel endpoint where to connect to
-PEER=
-
-# The local IP address of the tunnel endpoint
-LOCAL_ADDRESS=
+DEFAULT_MODE="gre"
 
 hook_check_settings() {
        assert isset MODE && assert isoneof MODE ${SUPPORTED_IP_TUNNEL_MODES}
index 1b4c3c023e0ce0a169da67f8876d22645832c066..50d43c75b15807f78b90fd58e30e91d972ac2867 100644 (file)
@@ -27,47 +27,37 @@ MODEM_ALLOWED_AUTH_METHODS="${PPP_ALLOWED_AUTH_METHODS}"
 HOOK_SETTINGS="HOOK"
 
 # Access Point Name.
-APN=
 HOOK_SETTINGS="${HOOK_SETTINGS} APN"
 
 # Sets the authentication algortihm that must be used.
-AUTH=
 HOOK_SETTINGS="${HOOK_SETTINGS} AUTH"
 
 # Baudrate.
-BAUDRATE=921600
+DEFAULT_BAUDRATE=921600
 HOOK_SETTINGS="${HOOK_SETTINGS} BAUDRATE"
 
 # The device name of the serial device.
 # XXX how can we make sure that this does not change all the time?
-DEVICE=
 HOOK_SETTINGS="${HOOK_SETTINGS} DEVICE"
 
 # A monitor device.
 # Send AT commands to this device, when the primary device is
 # connected.
-MONITOR_DEVICE=
 HOOK_SETTINGS="${HOOK_SETTINGS} MONITOR_DEVICE"
 
 # Maximum transmission unit.
-MTU=
 HOOK_SETTINGS="${HOOK_SETTINGS} MTU"
 
 # User credentials.
-USERNAME=
-PASSWORD=
 HOOK_SETTINGS="${HOOK_SETTINGS} USERNAME PASSWORD"
 
 # PIN code.
-PIN=
 HOOK_SETTINGS="${HOOK_SETTINGS} PIN"
 
 # Phone number.
-PHONE_NUMBER=
 HOOK_SETTINGS="${HOOK_SETTINGS} PHONE_NUMBER"
 
 # IMSI
-IMSI=
 HOOK_SETTINGS="${HOOK_SETTINGS} IMSI"
 
 hook_check_settings() {
index e113c92ea9751198db7d6a9cea58ab488a32da8c..cd3913b62034ff38a474d2a711070fcd3f0f059c 100644 (file)
 HOOK_SETTINGS="HOOK ACCESS_CONCENTRATOR AUTH USERNAME PASSWORD"
 HOOK_SETTINGS="${HOOK_SETTINGS} SERVICE_NAME MTU IPV6 PREFIX_DELEGATION"
 
-# User credentials for the dialin.
-USERNAME=""
-PASSWORD=""
-
-# Set the authentication mechanism.
-AUTH=
-
-# Access Concentrator.
-ACCESS_CONCENTRATOR=""
-
-# Service name.
-SERVICE_NAME=""
-
-# Maximum Transmission Unit.
-MTU=
-
 # This hook can work with all authentication methods supported by pppd.
 PPPOE_SUPPORTED_AUTH_METHODS="${PPP_SUPPORTED_AUTH_METHODS}"
 PPPOE_PLUGIN="rp-pppoe.so"
 
 # Request an IPv6 address.
-IPV6="true"
+DEFAULT_IPV6="true"
 
 # Use IPv6 prefix delegation.
-PREFIX_DELEGATION="true"
+DEFAULT_PREFIX_DELEGATION="true"
 
 hook_check_settings() {
        assert isset USERNAME