client-facing keep-alive idle value is too low

[bam80]

Как говорилось в #526 (см. также там ссылки на другое репорты), низкие значения начальной задержки для keep-alive пакетов может спровоцировать серьезные проблемы с разрядом аккумулятора.
С другой стороны, выбранное на данный момент зашитое в коде значение в 30 сек неоправданно мало - в моём NAT время "протухания" TCP соединения - 10 минут (выявлено экспериментально), и мне попадались другие репотры с таким же временем. О случаях с более короткими задержками в NAT мне ничего неизвестно.

Предлагаю как минимум увеличить значение keep-alive idle, зашитое в коде, до максимально необходимого значения, приближенного к реальности, например 590 сек.

См. также:
#441
#516

@dolonet

[dolonet]

bam80 is right that this isn't configurable on the client side. The accepted client socket is tuned in internal/utils/net_listener.go via the v1 network.SetClientSocketOptions, which hardcodes Idle=30s/Interval=10s/Count=3 (network/sockopts.go, network/init.go). The [network.keep-alive] config only reaches the outbound dialer (network/v2, built in makeNetwork) — it never touches the client listener, so there's no config workaround today.

@9seconds — two related calls here, both yours: (1) raise the hardcoded client-facing idle from 30s (bam80 measures NAT staleness ~10 min, and 30s probes interact badly with a mobile-modem firmware sleep bug → battery), and/or (2) make the client listener honor [network.keep-alive] the way the outbound path already does, instead of hardcoding. Happy to PR whichever shape you prefer.