app.js: 'consumer' is not defined

Dirvann / mediasoup-sfu-webrtc-video-rooms

A simple video conferencing example using the mediasoup sfu

Apache License 2.0

222 stars 78 forks source link

app.js: 'consumer' is not defined #19

Open bra1n opened 2 years ago

bra1n commented 2 years ago

https://github.com/Dirvann/mediasoup-sfu-webrtc-video-rooms/blob/dd52d6c932b21add48456f93dbba19a70f7e6a0c/src/app.js#L192

It seems like the server-side app.js is using an undefined consumer variable in the "resume" socket event listener. Is this an oversight or a bug?

bra1n commented 2 years ago

Ah, seems to be like this code is not even used. Is there a particular reason why resume is never called? IIRC, resuming the consumer is supposed to be necessary...