openconnect: fix token_script error handling/logging

When token_script fails, proto_setup_failed() notifies netifd that
setup has failed, but script execution continues regardless.
This commit adds exit 1 and sleep 5 (consistent with wireguard.sh
error handling) to abort setup on token_script failure. Also fix
typo in logger tag ("openconenct" -> "openconnect").

Signed-off-by: Thomas Winkler <redacted>

diff --git a/net/openconnect/files/openconnect.sh b/net/openconnect/files/openconnect.sh
index 6ec8b6629633feb3035c542e4fe211964978786f..2e60fc8b536b57d2414b5ae4690b799c4695edbe 100755 (executable)
--- a/net/openconnect/files/openconnect.sh
+++ b/net/openconnect/files/openconnect.sh
@@ -149,8 +149,10 @@ proto_openconnect_setup() {
                }
                [ "$token_mode" = "script" ] && {
                        $token_script >> "$pwfile" 2> /dev/null || {
-                               logger -t openconenct "Cannot get password from script '$token_script'"
+                               logger -t openconnect "Cannot get password from script '$token_script'"
+                               sleep 5
                                proto_setup_failed "$config"
+                               exit 1
                        }
                }
                append_args --passwd-on-stdin