Minor Feature Suggestion

mediasoup libraries
1

Hi guys, just wanted to suggest that an error message be modified to be more descriptive:

Message:

Error: Channel request handler with ID c1064f4a-fd94-4274-9f73-9382a6eff000 already exists [method:transport.produce]
    at Channel.processMessage (Channel.js:199:37)
    at Socket.<anonymous> (Channel.js:69:34)
    at Socket.emit (node:events:514:28)
    at addChunk (node:internal/streams/readable:324:12)
    at readableAddChunk (node:internal/streams/readable:297:9)
    at Readable.push (node:internal/streams/readable:234:10)
    at Pipe.onStreamRead (node:internal/stream_base_commons:190:23)
    at Pipe.callbackTrampoline (node:internal/async_hooks:130:17)

Took me a very long time to figure out that this was happening because I was using pipeToRouter with an origin router and target router that were in the same worker (which I now realize is not supported according to the documentation!).

Hopefully this post is helpful to anyone that makes the same mistake I did in the future :slight_smile:

2

The error message cannot be made more descriptive because it happens at very low level, but anyway it’s saying everything it knows: you are calling transport.produce() with a id that already exists in same worker. Yes, agreed, the message doesn’t explain the limitation. But the docs so as you realized.

1 Like