fix_next

opnsense-plugin/www/rustproxy/+POST_INSTALL.pre

@@ -21,6 +21,13 @@ fi
 # therefore refresh configd before the MVC service controller can call them.
 /usr/sbin/service configd restart >/dev/null 2>&1 || true
 /usr/local/sbin/pluginctl -cq cache_flush >/dev/null 2>&1 || true
+
+# Apply model migrations before rendering the service template.  In particular
+# 1.4.0 copies the old combined listener values into the split address/port
+# fields so upgrades keep their existing ports instead of falling back.
+if [ -x /usr/local/opnsense/mvc/script/run_migrations.php ]; then
+    /usr/local/opnsense/mvc/script/run_migrations.php >/dev/null 2>&1 || true
+fi
 /usr/local/sbin/configctl template reload OPNsense/RustProxy >/dev/null 2>&1 || true
 
 exit 0

opnsense-plugin/www/rustproxy/Makefile

@@ -1,6 +1,6 @@
 PLUGIN_NAME= rustproxy
 PLUGIN_VERSION?= 0.23.3
-PLUGIN_REVISION?= 0
+PLUGIN_REVISION?= 1
 PLUGIN_COMMENT= RustProxy reverse proxy service
 PLUGIN_MAINTAINER?= local@localhost
 PLUGIN_WWW?= https://git.linuxiarz.pl/gru/RustProxy

opnsense-plugin/www/rustproxy/src/etc/rc.d/rustproxy

@@ -160,7 +160,9 @@ fi
 command_args="-r -R ${rustproxy_restart_delay} -P ${pidfile} -p ${child_pidfile} ${rustproxy_daemon_user_arg} -o ${rustproxy_log_file} ${rustproxy_command} ${rustproxy_runtime_args}"
 required_files="${rustproxy_command}"
 start_precmd="rustproxy_prestart"
+start_postcmd="rustproxy_poststart"
 stop_postcmd="rustproxy_poststop"
+status_cmd="rustproxy_status"
 
 rustproxy_prestart()
 {
@@ -179,6 +181,72 @@ rustproxy_prestart()
     /bin/chmod 0640 "${rustproxy_log_file}"
 
     echo "RustProxy listeners: HTTP=${rustproxy_proxy_listen_resolved} TLS=${rustproxy_tls_listen_resolved} ADMIN=${rustproxy_admin_listen_resolved}"
+    echo "RustProxy exec: ${rustproxy_command} ${rustproxy_runtime_args}"
+    printf '%s\n' "${rustproxy_command} ${rustproxy_runtime_args}" > "${rustproxy_run_dir}/command.txt"
+    /bin/chmod 0640 "${rustproxy_run_dir}/command.txt"
+}
+
+rustproxy_child_pid()
+{
+    [ -r "${child_pidfile}" ] || return 1
+    _rp_pid="$(/bin/cat "${child_pidfile}" 2>/dev/null)"
+    case "${_rp_pid}" in
+        ''|*[!0-9]*) return 1 ;;
+    esac
+    /bin/kill -0 "${_rp_pid}" 2>/dev/null || return 1
+    printf '%s\n' "${_rp_pid}"
+}
+
+rustproxy_poststart()
+{
+    # daemon(8) may stay alive with -r even when rust-proxy exits immediately.
+    # Verify the supervised child so service/configd cannot report a false OK.
+    /bin/sleep 1
+    if _rp_pid="$(rustproxy_child_pid)"; then
+        echo "RustProxy child started as PID ${_rp_pid}"
+        return 0
+    fi
+
+    echo "RustProxy failed to stay running; stopping the restart supervisor." >&2
+    if [ -s "${rustproxy_log_file}" ]; then
+        echo "Last RustProxy log lines:" >&2
+        /usr/bin/tail -n 40 "${rustproxy_log_file}" >&2
+    fi
+    if [ -r "${pidfile}" ]; then
+        _rp_supervisor_pid="$(/bin/cat "${pidfile}" 2>/dev/null)"
+        case "${_rp_supervisor_pid}" in
+            ''|*[!0-9]*) ;;
+            *) /bin/kill -TERM "${_rp_supervisor_pid}" 2>/dev/null || true ;;
+        esac
+    fi
+    /bin/sleep 1
+    /bin/rm -f "${child_pidfile}" "${pidfile}"
+    return 1
+}
+
+rustproxy_status()
+{
+    if _rp_pid="$(rustproxy_child_pid)"; then
+        echo "rustproxy is running as pid ${_rp_pid}"
+        return 0
+    fi
+
+    _rp_supervisor_pid=""
+    if [ -r "${pidfile}" ]; then
+        _rp_supervisor_pid="$(/bin/cat "${pidfile}" 2>/dev/null)"
+    fi
+    case "${_rp_supervisor_pid}" in
+        ''|*[!0-9]*) ;;
+        *)
+            if /bin/kill -0 "${_rp_supervisor_pid}" 2>/dev/null; then
+                echo "rustproxy is not running (daemon supervisor ${_rp_supervisor_pid} is waiting/restarting)"
+                return 1
+            fi
+            ;;
+    esac
+
+    echo "rustproxy is not running"
+    return 1
 }
 
 rustproxy_poststop()

opnsense-plugin/www/rustproxy/src/opnsense/mvc/app/controllers/OPNsense/RustProxy/Api/ServiceController.php

@@ -13,6 +13,8 @@ class ServiceController extends ApiMutableServiceControllerBase
 
     protected function reconfigureForceRestart()
     {
-        return 0;
+        // Listener addresses are process startup arguments, so Apply must
+        // always stop and start the service after regenerating the template.
+        return 1;
     }
 }

opnsense-plugin/www/rustproxy/src/opnsense/mvc/app/controllers/OPNsense/RustProxy/forms/general.xml

@@ -20,13 +20,13 @@
         <id>rustproxy.general.proxyAddress</id>
         <label>Bind address</label>
         <type>text</type>
-        <help>HTTP bind address. Leave empty for 0.0.0.0 or to keep the legacy combined listener after an upgrade.</help>
+        <help>HTTP bind address. Default: 0.0.0.0. Existing installations are migrated from the legacy combined listener automatically.</help>
     </field>
     <field>
         <id>rustproxy.general.proxyPort</id>
         <label>Port</label>
         <type>text</type>
-        <help>HTTP port passed as --proxy-listen. Leave empty for 8080 or to keep the legacy combined listener after an upgrade.</help>
+        <help>HTTP port passed as --proxy-listen. Default: 8080.</help>
     </field>
 
     <field>
@@ -44,13 +44,13 @@
         <id>rustproxy.general.tlsAddress</id>
         <label>Bind address</label>
         <type>text</type>
-        <help>HTTPS/TLS bind address. Leave empty for 0.0.0.0 or to keep the legacy combined listener after an upgrade.</help>
+        <help>HTTPS/TLS bind address. Default: 0.0.0.0. Existing installations are migrated automatically.</help>
     </field>
     <field>
         <id>rustproxy.general.tlsPort</id>
         <label>Port</label>
         <type>text</type>
-        <help>HTTPS/TLS port passed as --tls-listen. Leave empty for 8181 or to keep the legacy combined listener after an upgrade.</help>
+        <help>HTTPS/TLS port passed as --tls-listen. Default: 8181.</help>
     </field>
 
     <field>
@@ -62,13 +62,13 @@
         <id>rustproxy.general.adminAddress</id>
         <label>Bind address</label>
         <type>text</type>
-        <help>Administration bind address. Leave empty for 0.0.0.0 or to keep the legacy combined listener after an upgrade.</help>
+        <help>Administration bind address. Default: 0.0.0.0. Existing installations are migrated automatically.</help>
     </field>
     <field>
         <id>rustproxy.general.adminPort</id>
         <label>Port</label>
         <type>text</type>
-        <help>Administration port passed as --admin-listen. Leave empty for 9090 or to keep the legacy combined listener after an upgrade.</help>
+        <help>Administration port passed as --admin-listen. Default: 9090.</help>
     </field>
     <field>
         <id>rustproxy.general.allowRemoteAdmin</id>

opnsense-plugin/www/rustproxy/src/opnsense/mvc/app/models/OPNsense/RustProxy/Migrations/M1_4_0.php

@@ -0,0 +1,68 @@
+<?php
+
+namespace OPNsense\RustProxy\Migrations;
+
+use OPNsense\Base\BaseModelMigration;
+
+/**
+ * Repair the 1.3.x split-listener transition.
+ *
+ * 1.2.x stored listeners as a single IP:port field.  The first 1.3.x package
+ * introduced address/port fields without model defaults, so an upgrade could
+ * save fallback/new values while the still-valid legacy values remained in
+ * config.xml.  Copy the legacy values once into the split fields so existing
+ * installations keep the exact ports they used before the upgrade.
+ */
+class M1_4_0 extends BaseModelMigration
+{
+    private function splitListener($value)
+    {
+        $value = trim((string)$value);
+        if ($value === '') {
+            return null;
+        }
+
+        if (preg_match('/^\[([^\]]+)\]:(\d+)$/', $value, $matches)) {
+            $address = $matches[1];
+            $port = $matches[2];
+        } else {
+            $separator = strrpos($value, ':');
+            if ($separator === false) {
+                return null;
+            }
+            $address = substr($value, 0, $separator);
+            $port = substr($value, $separator + 1);
+        }
+
+        if ($address === '' || !ctype_digit($port)) {
+            return null;
+        }
+        $number = (int)$port;
+        if ($number < 1 || $number > 65535) {
+            return null;
+        }
+
+        return [$address, (string)$number];
+    }
+
+    private function migrateListener($general, $legacyField, $addressField, $portField)
+    {
+        $parts = $this->splitListener((string)$general->$legacyField);
+        if ($parts === null) {
+            return;
+        }
+        $general->$addressField = $parts[0];
+        $general->$portField = $parts[1];
+    }
+
+    public function run($model)
+    {
+        parent::run($model);
+
+        // Deliberately prefer valid 1.2.x values during this one migration.
+        // This also repairs hosts that already installed the broken 1.3.x model.
+        $this->migrateListener($model->general, 'proxyListen', 'proxyAddress', 'proxyPort');
+        $this->migrateListener($model->general, 'tlsListen', 'tlsAddress', 'tlsPort');
+        $this->migrateListener($model->general, 'adminListen', 'adminAddress', 'adminPort');
+    }
+}

opnsense-plugin/www/rustproxy/src/opnsense/mvc/app/models/OPNsense/RustProxy/RustProxy.xml

@@ -1,6 +1,6 @@
 <model>
     <mount>//OPNsense/rustproxy</mount>
-    <version>1.3.0</version>
+    <version>1.4.0</version>
     <description>RustProxy service settings</description>
     <items>
         <general>
@@ -17,10 +17,14 @@
                 fields below for upgrade compatibility, but new UI writes these.
             -->
             <proxyAddress type="NetworkField">
+                <Default>0.0.0.0</Default>
+                <Required>Y</Required>
                 <NetMaskAllowed>N</NetMaskAllowed>
                 <ValidationMessage>Enter a valid HTTP bind address, for example 0.0.0.0.</ValidationMessage>
             </proxyAddress>
             <proxyPort type="PortField">
+                <Default>8080</Default>
+                <Required>Y</Required>
                 <ValidationMessage>Enter a valid HTTP port between 1 and 65535.</ValidationMessage>
             </proxyPort>
             <tlsEnabled type="BooleanField">
@@ -28,17 +32,25 @@
                 <Required>Y</Required>
             </tlsEnabled>
             <tlsAddress type="NetworkField">
+                <Default>0.0.0.0</Default>
+                <Required>Y</Required>
                 <NetMaskAllowed>N</NetMaskAllowed>
                 <ValidationMessage>Enter a valid HTTPS/TLS bind address, for example 0.0.0.0.</ValidationMessage>
             </tlsAddress>
             <tlsPort type="PortField">
+                <Default>8181</Default>
+                <Required>Y</Required>
                 <ValidationMessage>Enter a valid HTTPS/TLS port between 1 and 65535.</ValidationMessage>
             </tlsPort>
             <adminAddress type="NetworkField">
+                <Default>0.0.0.0</Default>
+                <Required>Y</Required>
                 <NetMaskAllowed>N</NetMaskAllowed>
                 <ValidationMessage>Enter a valid administration bind address, for example 0.0.0.0.</ValidationMessage>
             </adminAddress>
             <adminPort type="PortField">
+                <Default>9090</Default>
+                <Required>Y</Required>
                 <ValidationMessage>Enter a valid administration port between 1 and 65535.</ValidationMessage>
             </adminPort>
             <allowRemoteAdmin type="BooleanField">