signalapp / Signal-Android

A private messenger for Android.
https://signal.org
GNU Affero General Public License v3.0
25.65k stars 6.16k forks source link

Unable to open AVI files directly from the Signal app #13694

Open Anvil5465 opened 2 months ago

Anvil5465 commented 2 months ago

Bug description

AVI video files open properly when opened from the files app but not when opened via signal. iOS users can open the file just fine.

Steps to reproduce

Actual result: error "Can't find an app able to open this media." Expected result: signal uses VLC to play the video file

Screenshots

signal-2024-09-15-08-11-33-415

Device info

Device: Google Pixel 6a Android version: 14 (GrapheneOS 2024090400) Signal version: 7.16.4

Link to debug log

[signal-log-1726402004259.zip](https://github.com/user-attachments/files/17005720/signal-log-1726402004259.zip)
Anvil5465 commented 2 months ago

I can also attach the AVI file that is in the screenshot if that would be helpful.

cody-signal commented 2 months ago

Looks like we don't detect the file as a video and then try to use a 3rd part app to play which the OS also doesn't find one for. Not much else we can do here. We can only render videos the OS can play for us, so maybe a Graphene thing?

Anvil5465 commented 2 months ago

If that's the case then wouldn't the files app also be unable to open the same AVI file using VLC? As you can see in the screenshot below, the OS definitely recognizes VLC as an app capable of opening AVI files.

Screenshot_20240916-120828

cody-signal commented 2 months ago

:shrug:

We take the info we know about the file and give it to the OS. It's up to the OS to tells us if there's an app available to play it or not.

Anvil5465 commented 2 months ago

Nextcloud talk is also able to get to the same interface for opening the AVI file as the files app

stale[bot] commented 1 day ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.