Repository navigation

mod-audio-fork: robustness fixes (lifetime, races, input validation, tests) - #3

Draft

pragmatrix wants to merge 36 commits into

questnetfrom

robustness-fixes

Draft

mod-audio-fork: robustness fixes (lifetime, races, input validation, tests)#3
pragmatrix wants to merge 36 commits into
questnetfrom
robustness-fixes

Commits on Oct 8, 2026
- mod-audio-fork: raise start metadata and send_text limit to 1 MB, queue texts on heap
  pragmatrix
  committed
  0f7bf4c
- mod-audio-fork: queue texts sent before the websocket is connected
  pragmatrix
  committed
  6a917d7
- mod-audio-fork: add unit tests for base64 and vector_math
  pragmatrix
  committed
  09110aa
- repo: add Dockerfile compiling mod_audio_fork against FreeSWITCH 1.11.3 headers
  pragmatrix
  committed
  cc2d844
- repo: add justfile for docker-based test targets
  pragmatrix
  committed
  610115f
- mod-audio-fork: add Audio-Fork-Bugname header to all fired events
  pragmatrix
  committed
  2911160
- mod-audio-fork: store bugname in session pool to remove 64 byte limit
  pragmatrix
  committed
  15d1d6b
- mod-audio-fork: fix start arg validation, playAudio NULL type, stereo resample overflow, null logs
  
  pragmatrix
  and
  claude
  committed
  c076fa7
- mod-audio-fork: build with c++17
  
  pragmatrix
  and
  claude
  committed
  c97c312
- mod-audio-fork: own AudioPipe via shared_ptr to fix use-after-free
  
  pragmatrix
  and
  claude
  committed
  1958a76
- mod-audio-fork: wrap session-side AudioPipe reference in PipeHandle
  
  pragmatrix
  and
  claude
  committed
  c64605f
- mod-audio-fork: fix races between lws thread and session cleanup, drop playAudio and transcription
  
  pragmatrix
  and
  claude
  committed
  f0746c6
- issues: add review findings with current status
  
  pragmatrix
  and
  claude
  committed
  204ec0f
- mod-audio-fork: add playback mode that skips forking caller audio
  
  pragmatrix
  and
  claude
  committed
  44baf2a
- mod-audio-fork: close pipes that are not yet connected, clean up early start failures
  
  pragmatrix
  and
  claude
  committed
  f8517cc
- mod-audio-fork: terminate copied strings, reject over-long host/path, drop unused private_t fields
  
  pragmatrix
  and
  claude
  committed
  983a7e0
- mod-audio-fork: only accept inbound binary when audio is actually played back
  
  pragmatrix
  and
  claude
  committed
  d12266e
- mod-audio-fork: saturate instead of wrap when mixing playout audio into the frame
  
  pragmatrix
  and
  claude
  committed
  2b15af1
- mod-audio-fork: wake the lws service thread on unload so it can exit
  
  pragmatrix
  and
  claude
  committed
  a8d1bd0
- mod-audio-fork: remove unused mapMutex
  
  pragmatrix
  and
  claude
  committed
  d343675
- ide: add just ide-setup to fetch FreeSWITCH/lws headers and generate clangd and IntelliSense config
  
  pragmatrix
  and
  claude
  committed
  d669d89
- mod-audio-fork: hold the session mutex when send_text and graceful_shutdown read the pipe handle
  
  pragmatrix
  and
  claude
  committed
  82fea7a
- issues: add the /code-review findings to review-findings.md
  
  pragmatrix
  and
  claude
  committed
  a9cb19e
- mod-audio-fork: require the sample rate argument for start instead of reading a NULL argv
  
  pragmatrix
  and
  claude
  committed
  a263446
- issues: mark the fork_session_cleanup early return as checked, not an issue
  
  pragmatrix
  and
  claude
  committed
  6b574ad
- mod-audio-fork: check the read codec after pre-answer before starting
  
  pragmatrix
  and
  claude
  committed
  41fb1fe
- mod-audio-fork: catch C++ exceptions before they reach lws and the C callers, check read_impl status
  
  pragmatrix
  and
  claude
  committed
  fe6603d
- issues: record the second review pass, close the refuted findings
  
  pragmatrix
  and
  claude
  committed
  05875ee
- mod-audio-fork: limit the start sample rates to 64000
  
  pragmatrix
  and
  claude
  committed
  beb452d
- issues: mark the sample rate limit as fixed
  
  pragmatrix
  and
  claude
  committed
  a045e16
- mod-audio-fork: check tech_pvt for null before reading its id in fork_session_cleanup
  pragmatrix
  committed
  5278835
- issues: mark the wakeup and tech_pvt id items as checked
  pragmatrix
  committed
  2bdd948
- ci: add qa workflow running the docker tests with cancel-in-progress
  pragmatrix
  committed
  fc894e3
- ci: run qa on pushes to questnet and on pull requests only
  pragmatrix
  committed
  b21a984
- issues: add upstream watch list and AGENTS.md pointer
  
  pragmatrix
  and
  claude
  committed
  2360387
- issues: add kriklivsky, TopCS and the fork network sweep to upstream-watch, note the receive buffer and event subclass findings
  
  pragmatrix
  and
  claude
  committed
  904a095

Commits

Commits on Oct 8, 2026