signalapp / Signal-Android

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

Unable to send .djvu file #13233

Open johanw666 opened 11 months ago

johanw666 commented 11 months ago

Bug description

When I try to send a .djvu file (a format mostly used for scanned books because it is often much smaller than pdf) Signal gives the error that it can’t send the "image".

Steps to reproduce

Actual result: An error message pops up Expected result: The .djvu file is sent as a file attachment.

Screenshots

See forum thread: https://community.signalusers.org/t/unable-to-send-djvu-file/56511

Device info

Device: Samsung A50 Android version: 11 Signal version: 6.37.1

Link to debug log

https://debuglogs.org/android/6.37.1-JW/4b96194a9b30ea4ba1236883119a4f8cf12dd71bf74bd44df85d7f5c606bd674

Keisial commented 10 months ago

MediaUtil.isImageType() is already excluding "image/svg+xml" (commit b94624fd5aa230c59c6c7bf5823f16d316406943). A quick solution might be to similarly exclude image/vnd.djvu despite not seeming a proper solution.

However, it is already catching the exception and returning a 0x0 Pair...

stale[bot] commented 8 months 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.

johanw666 commented 8 months ago

It is still an issue, please track.

johanw666 commented 7 months ago

MediaUtil.isImageType() is already excluding "image/svg+xml" (commit b94624f). A quick solution might be to similarly exclude image/vnd.djvu despite not seeming a proper solution.

However, it is already catching the exception and returning a 0x0 Pair...

I tried adding image/vnd.djvu to that list. Android tried to send it but it got a send error so that can't be the only part of the solution.