Iñaki Baz Castillo
Iñaki Baz Castillo
Fixed here: https://github.com/versatica/mediasoup/commit/dedfb24a6a5c5c25ee91744b10bc426e3dbdf5cc Thanks @satoren!
### Issue 1: test-node-sctp.ts fails I've added some console logs: https://github.com/versatica/mediasoup/pull/1353/commits/069f78e62c618afa37622e119b4ea0d522fc9da1 The test fails because first sent message is not later received by the data consumer: ``` npx jest --testPathPattern...
Ok, so I've solved the problem as follows: https://github.com/versatica/mediasoup/pull/1353/commits/1a25bedd503f2a96f3d745f7f9af203e02688ab9 Test now works (for simplicity is temporary reduced to just 2 messages) and all logs are good: ``` DepUsrSCTP::SendSctpDataStore() | ----------...
And **OF COURSE** a double free bug that just happens in Rust in CI in Ubuntu and not in MacOS, to make me spend yet another 2 extra days on...
> Are you making sure that threadA is not writing to a `SendSctpDataStore` at the when at the same time the `uv` main thread is reading it? ClassDestroy() method should...
Also take into account that this PR doesn't yet implement the thing about running the Checker in a separate thread.
> > Are you making sure that threadA is not writing to a `SendSctpDataStore` at the when at the same time the `uv` main thread is reading it? > >...
I've done many changes in https://github.com/versatica/mediasoup/pull/1353/commits/ca5f222e914cde25622de368ca927a9b3dbaf1a8: 1. Move creation of `DepUsrSCTP::Checker` from `Worker` constructor/destructor to `DepUsrSCTP::ClassInit()` and `DepUsrSCTP::ClassDestroy()`. This comes with an issue (see **Issue 2** in next comment). 2....
### Issue 2 When running any test in Node: ```bash npx jest --testPathPattern node/src/test/test-node-sctp.ts ``` ``` mediasoup:ERROR:Worker (stderr) DepLibUV::onWalk() | alive UV handle found (this shouldn't happen) [type:timer, active:0, closing:1,...
After a meeting with Jose we have decided that, as originally planned, the timer of the usrsctp `Checker` singleton must run in a separate thread X and when the first...