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.

Sign up for GitHub

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

Jump to bottom

[Bug|Crash] Voice Messages sending need more robustfully #3833

Closed
DoM1niC opened this issue Aug 14, 2021 · 1 comment · Fixed by #4547
Closed

[Bug|Crash] Voice Messages sending need more robustfully #3833

DoM1niC opened this issue Aug 14, 2021 · 1 comment · Fixed by #4547
Assignees
@onurays @ouchadam
Labels
A-Voice-Messages T-Defect Something isn't working: bugs, crashes, hangs and other reported problems

Comments

@DoM1niC
Copy link

DoM1niC commented Aug 14, 2021 •
edited
Loading

Hey Element Devs,

The Issue cause by low or unstable internet connectivity, the Voice Message temporarily dismiss and try to resend but the tmp file is broken or seems to have invalid data, the 2nd problem after some retries the App crash instantly by opening this Room again.

Code: Develop
Server: Synapse last Stable 1.43.0 (messenger.3dns.eu)

send-error.mp4
@DoM1niC DoM1niC mentioned this issue Aug 14, 2021
Open
14 tasks
@DoM1niC DoM1niC changed the title [Bug] Voice Messages recording don't send correctly [Bug|Crash] Voice Messages sending need more robustfully Sep 27, 2021
@bmarty bmarty added A-Voice-Messages T-Defect Something isn't working: bugs, crashes, hangs and other reported problems labels Sep 27, 2021
@onurays onurays self-assigned this Sep 28, 2021
@onurays onurays mentioned this issue Sep 28, 2021
Merged
@DoM1niC DoM1niC mentioned this issue Nov 19, 2021
Closed
ouchadam added a commit that referenced this issue Nov 22, 2021
@ouchadam
fixing #3833, it's caused by us always using the same file name when …
6e8c545 
…starting a recording, which means any previous unsent recordings get deleted
@ouchadam ouchadam self-assigned this Nov 22, 2021
@DoM1niC
Copy link
Author

DoM1niC commented Nov 22, 2021

Fixed by #4527

@ouchadam ouchadam mentioned this issue Nov 23, 2021
Merged
@ouchadam ouchadam mentioned this issue Nov 24, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@onurays onurays

@ouchadam ouchadam

Labels
A-Voice-Messages T-Defect Something isn't working: bugs, crashes, hangs and other reported problems
Projects
None yet
Milestone
No milestone
4 participants
@onurays @DoM1niC @ouchadam @bmarty