-
Notifications
You must be signed in to change notification settings - Fork 731
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
Voice recording crashes on Android 10+ #6989
Comments
Can confirm, pressing the voice record button and quickly tapping it again throws the expected exception. The referenced voice message file is empty (size: 0B). |
It's also preceded by:
|
I can also reproduce it some times if the recording gets into an inconsistent state and I lock the device or send the app to background, but I don't really know what causes it. It does look like a race condition of some kind. |
Fixed by #7009 I guess. |
Steps to reproduce
The reason is unknown at the moment, but an empty file could cause it. I'm guessing it's either a too short voice message which didn't have its data flushed or some kind of path or issue with the logic that converts the uri into the file to read and send it.
It's not related to the Opus library since it only happens on Android versions >= 10, and
VoiceRecorderQ
uses the Opus encoder included in the OS.Outcome
What did you expect?
The voice recording should work and be sent normally.
What happened instead?
It crashes with the error message:
Your phone model
Several
Operating system version
10+
Application version and app store
1.4.31, 1.4.32, 1.4.34
Homeserver
matrix.org
Will you send logs?
No
Are you willing to provide a PR?
Yes
The text was updated successfully, but these errors were encountered: