fix: improve connection pool idle and max age checks to allow disabling #3

2026-06-08T16:55:46-04:00

serversdown commented

2026-06-08 16:55:46 -04:00

No description provided.

serversdown added 10 commits 2026-06-08 16:55:47 -04:00

fix: improve connection pool idle and max age checks to allow disabling af5ecc1a92

fix: change debug logs to info level for connection pool events a302fd15d4

feat: add connection pool status logging every 15 minutes 01337696b3

chore: gitignore dev data 996b993cb9

Merge branch 'dev-persistent' of ssh://10.0.0.2:2222/serversdown/slmm into dev-persistent 0a7422eceb

chore: data-dev folder added to gitignore 98a8d357e5

chore: gitignore clean up fefa9eace8

stop tracking dev runtime data 450509d210

fix: use request-first TemplateResponse signature e3f9ca7f5b

Modern Starlette requires `request` as the first positional arg to
TemplateResponse. The old `TemplateResponse(name, context)` form caused
the context dict to be passed as the template name, which Jinja2 then
tried to use as a cache key -> TypeError: unhashable type: 'dict' (500
on GET / and /roster).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

fix: reuse pooled TCP connection for DRD streaming af86cf713e

stream_drd() discarded the pooled connection and forced a fresh connect.
The NL43 allows only one TCP connection at a time; over a cellular link
the device does not free its single slot fast enough for an immediate
reconnect, so the fresh connect times out — the live DRD stream fails
while start/stop commands (which reuse the warm pooled socket) keep
working. This surfaced once the persistent connection pool was enabled
(TCP_PERSISTENT_ENABLED=true).

Stream over the already-open pooled connection via acquire() instead of
discard()+_open_connection(), and release() it back to the pool on exit
(after sending SUB to stop the stream) so commands keep reusing the same
single socket. The per-device lock is held for the whole streaming
session, so the poller can't touch the socket concurrently.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

serversdown merged commit d6dd2e736b into main

2026-06-08 16:56:34 -04:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: serversdown/slmm#3