Description of changes: I'm not sure why we aren't sending receiver reports if audio/video attendee capability are None but it doesn't really matter too much. I just made this check more resilient by also checking stun packets, which should be sent every 5 seconds as well.
Testing:
Can these tested using a demo application? Please provide reproducible step-by-step instructions.
Send audio/video attendee capabilities to None
Join meeting and ensure you are not disconnected after 15 seconds.
Checklist:
Have you successfully run npm run build:release locally?
y
Do you add, modify, or delete public API definitions? If yes, has that been reviewed and approved?
n
Do you change the wire protocol, e.g. the request method? If yes, has that been reviewed and approved?
n
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Issue #: N/A
Description of changes: I'm not sure why we aren't sending receiver reports if audio/video attendee capability are None but it doesn't really matter too much. I just made this check more resilient by also checking stun packets, which should be sent every 5 seconds as well.
Testing:
Can these tested using a demo application? Please provide reproducible step-by-step instructions.
Checklist:
Have you successfully run
npm run build:releaselocally? yDo you add, modify, or delete public API definitions? If yes, has that been reviewed and approved? n
Do you change the wire protocol, e.g. the request method? If yes, has that been reviewed and approved? n
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.