diff options
| author | Felix Fietkau | 2026-02-06 10:12:31 +0000 |
|---|---|---|
| committer | Felix Fietkau | 2026-02-07 09:04:18 +0000 |
| commit | 212040b5cac619ec8009b0b4262d11b4e3abfdc4 (patch) | |
| tree | a0f731c45629378c2f0b00cf906c92cdc2868ede | |
| parent | 389a79d97230fbd31de98a7f8896302ad284c114 (diff) | |
| download | openwrt-212040b5cac619ec8009b0b4262d11b4e3abfdc4.tar.gz | |
unetmsg: fix reconnect loop when RX authenticates before TX
When both peers connect simultaneously, the RX side can authenticate
before the TX handshake completes. network_check_auth() was sending a
ping on the unauthenticated TX channel, which gets rejected by the
remote's pre-auth handler as "Auth failed", killing the connection and
triggering an endless reconnect cycle.
Check chan.auth before interacting with the TX channel. If TX auth
hasn't completed yet, just schedule a reconnect timer - auth_data_cb
already handles state sync when TX auth completes.
Signed-off-by: Felix Fietkau <nbd@nbd.name>
| -rw-r--r-- | package/network/services/unetmsg/files/usr/share/ucode/unetmsg/unetmsgd-remote.uc | 10 |
1 files changed, 1 insertions, 9 deletions
diff --git a/package/network/services/unetmsg/files/usr/share/ucode/unetmsg/unetmsgd-remote.uc b/package/network/services/unetmsg/files/usr/share/ucode/unetmsg/unetmsgd-remote.uc index f6b0ef2d59..c7301e1a09 100644 --- a/package/network/services/unetmsg/files/usr/share/ucode/unetmsg/unetmsgd-remote.uc +++ b/package/network/services/unetmsg/files/usr/share/ucode/unetmsg/unetmsgd-remote.uc @@ -203,16 +203,8 @@ function network_check_auth(sock_data, info) core.dbg(`Incoming connection from ${sock_data.name} established\n`); let chan = net.tx_channels[sock_data.name]; - if (!chan) { + if (!chan || !chan.auth) net.timer.set(100); - return; - } - - chan.channel.request({ - method: "ping", - data: {}, - return: "ignore", - }); } function network_accept(net, sock, addr) |