aws / amazon-chime-sdk-js

A JavaScript client library for integrating multi-party communications powered by the Amazon Chime service.
Apache License 2.0
709 stars 477 forks source link

Camera lights, leaving meetings, and console warnings #2794

Closed samspycher closed 11 months ago

samspycher commented 11 months ago

This line logs a console.warn message when an action occurs in a state that does not support a transition for the given action. It will log this warning if an Update happens in the Disconnecting state:

No transition found from Disconnecting with Update

I suspect this warning is occurring frequently in our case because of the way we leave meetings:

await  audioVideo.stopVideoInput();
await  audioVideo.stopAudioInput();
audioVideo.stop();

This feels paranoid, but I am doing it because the [documented way](https://aws.github.io/amazon-chime-sdk-js/modules/apioverview.html#:~:text=To%20stop%20the%20meeting%20session%2C%20call%20meetingSession.audioVideo.stop().) to end meetings is apparently incomplete, given [further documentation](https://aws.github.io/amazon-chime-sdk-js/modules/faqs.html#after-leaving-a-meeting-the-camera-led-is-still-on-indicating-that-the-camera-has-not-been-released-what-could-be-wrong:~:text=meetingSession.audioVideo.stopVideoInput()%3B) that seems justified given my experience and also several stackoverflow questions on the topic [1], [2] (bottom answer is where I take my code from).

I have two questions:

Note: From my perspective as a user of this API , I do think fully leaving meetings (and releasing all employed resources) should be a single line of code. Perhaps I am missing something? But if not, the percentage of users who are googling in frustration after chapter 4 left their camera light running has got to be close to 100%.

samspycher commented 11 months ago

Closing this, because I didn't observe the issue template. Apologies!

New issue is https://github.com/aws/amazon-chime-sdk-js/issues/2796.