Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

microphone not published on windows build #64

Closed
alex-bluetrain opened this issue Dec 27, 2021 · 3 comments · Fixed by #127
Closed

microphone not published on windows build #64

alex-bluetrain opened this issue Dec 27, 2021 · 3 comments · Fixed by #127

Comments

@alex-bluetrain
Copy link
Contributor

alex-bluetrain commented Dec 27, 2021

When building or running on windows, the localAudioTrack (microphone) doesn't get published.

it doesn't show any errors or warnings, it just doesn't work.

I've tried with LocalAudioTrack.create() + localParticipant.publishAudioTrack()
and also with just the localParticipant.setMicrophoneEnabled(true);

same results.

to reproduce: just run or build the example provided in client-sdk-flutter and check the published audio tracks on the room.

tested on 0.5.5

@jdde
Copy link
Contributor

jdde commented Mar 31, 2022

Hey @alex-bluetrain does the client-sdk-flutter already supports windows?
Based on the README.md in the master branch, it's not yet supported. (Work in progress)

Edit:
nevermind, basic support was merged in 0.5.5

@cloudwebrtc
Copy link
Contributor

cloudwebrtc commented Jun 3, 2022

I'm pretty sure that this bug is caused by the built-in AEC startup logic because some users in dart-sip-ua reported similar issues. when recording is started before playback, and Windows has built-in AEC enabled, the microphone will fail to start.

The PR of the relevant native SDK, then the flutter-webrtc needs to be updated.
webrtc-sdk/webrtc#23 => webrtc-sdk/webrtc#29

@cloudwebrtc
Copy link
Contributor

fixed #127

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants