Eyevinn / moqlivemock / 33518208352

53%

main: 53%

Build Type

Pull #136

github

Committed by tobbee

Commit Message

fix(mlmrel): register an announcement before acknowledging it

The publish-namespace handler sent REQUEST_OK and then registered the
namespace. A publisher that subscribes -- or tells a subscriber to --
the moment its PublishNamespace returns could race the handler
goroutine and get DOES_NOT_EXIST for a namespace the relay had just
acknowledged. TestFanoutSharedUpstream hit exactly that window on CI's
slower runners, and the interop announce-subscribe case has the same
shape.

Register first, answer second: a SUBSCRIBE that observed the OK now
always resolves. An Accept that fails (a broken request stream)
deregisters again.

Pull Request Pull Request #136: feat(mlmrel): add a MoQ relay with fanout, group cache and FETCH relaying

Coverage Stats

628 of 1118 new or added lines in 15 files covered. (56.17%)

3 existing lines in 2 files now uncovered.

3052 of 5795 relevant lines covered (52.67%)

1230.87 hits per line