diff options
| author | Florian Eckert <fe@dev.tdt.de> | 2020-11-04 16:24:55 +0100 |
|---|---|---|
| committer | Florian Eckert <fe@dev.tdt.de> | 2020-11-06 10:41:28 +0100 |
| commit | 1f1d21ed28c7e6713fb010e380b8557a9d020b16 (patch) | |
| tree | 6aeb4648e67000dd606e9708ed7781e3c6e9cca4 /net/mwan3/files/etc/hotplug.d | |
| parent | 59d12598b50edbc5a7a1f178c3c291524bd85b13 (diff) | |
mwan3: make it clearer who initiated the start
It was somewhat opaque how the variable a is questioned. To show this
better the variable is now a string and not a boolean. So you can see
directly what should happen. With a boolean you always have to think
about what it means when 0 or 1 is used.
Signed-off-by: Florian Eckert <fe@dev.tdt.de>
Diffstat (limited to 'net/mwan3/files/etc/hotplug.d')
| -rw-r--r-- | net/mwan3/files/etc/hotplug.d/iface/15-mwan3 | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/mwan3/files/etc/hotplug.d/iface/15-mwan3 b/net/mwan3/files/etc/hotplug.d/iface/15-mwan3 index 5fe3e33e0..f1755d26f 100644 --- a/net/mwan3/files/etc/hotplug.d/iface/15-mwan3 +++ b/net/mwan3/files/etc/hotplug.d/iface/15-mwan3 @@ -23,7 +23,7 @@ fi config_load mwan3 /etc/init.d/mwan3 running || { - [ "$MWAN3_STARTUP" = 1 ] || procd_lock + [ "$MWAN3_STARTUP" = "init" ] || procd_lock LOG notice "mwan3 hotplug $ACTION on $INTERFACE not called because globally disabled" mwan3_flush_conntrack "$INTERFACE" "$ACTION" exit 0 @@ -69,7 +69,7 @@ case "$ACTION" in mwan3_create_iface_iptables $INTERFACE $DEVICE mwan3_create_iface_rules $INTERFACE $DEVICE mwan3_set_iface_hotplug_state $INTERFACE "$status" - if [ "$MWAN3_STARTUP" != 1 ]; then + if [ "$MWAN3_STARTUP" = "cmd" ]; then mwan3_create_iface_route $INTERFACE $DEVICE mwan3_set_general_rules [ "$status" = "online" ] && mwan3_set_policies_iptables |