Eyevinn / moqlivemock / 33604230178

53%

Build Type

push

github

Committed by tobbee

Commit Message

feat(mlmrel): bound the wait for an upstream answer with -upstream-timeout

A forwarded SUBSCRIBE or a proxied FETCH waited for the upstream's answer for as long as the downstream requester cared to wait. imquic's interop client showed the cost: in its subscribe-before-announce case the publisher receives the forwarded SUBSCRIBE and never answers it, so the subscriber hung until the case timed out, although a REQUEST_ERROR would have passed it. The wait is now bounded by -upstream-timeout (default 5s; the interop entrypoint uses 1s), and on expiry the downstream request is rejected with TIMEOUT while the upstream request is cancelled. In moqtransport the context passed to Subscribe and Fetch governs only the request's establishment, so bounding it does not touch an established subscription or fetch. Two synctest tests pin the timing and the cancellation for both request types.

Coverage Stats

30 of 36 new or added lines in 4 files covered. (83.33%)

37 existing lines in 1 file now uncovered.

3082 of 5822 relevant lines covered (52.94%)

1225.76 hits per line