[webrtc-rtptransport] Should RTCRtpTransport(Processor) use event handlers for packet notifications? (#70)

youennf has just created a new issue for https://github.com/w3c/webrtc-rtptransport:

== Should RTCRtpTransport(Processor) use event handlers for packet notifications? ==
If `onpacketizedrtpavailable` is not set, I assume that packetised data is being sent to the network automatically.
On the contrary if `onpacketizedrtpavailable` is set, I assume that packetized data is not being sent automatically.

This is somehow problematic since setting event handlers should not have side effects like this.
Also, it is unclear what happens if event handlers are not set and one of the read packets method is called.

One possibility is to use `preventDefault` for events but this seems a bit cumbersome.
Also, given the idea is to call a read method, having several listeners is somehow wasteful/footgun.

I am wondering whether we should not switch to callbacks instead.

Something like:
processor.setPacketizedRtpHandler(() => {
});

We could directly pass all the packets as a parameter to remove the need for the read methods.

Please view or discuss this issue at https://github.com/w3c/webrtc-rtptransport/issues/70 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Wednesday, 4 September 2024 14:49:46 UTC