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

Unknown extension .cmap #7160

Closed
jomapp opened this issue Jul 3, 2024 · 1 comment · Fixed by #7312 or #7412
Closed

Unknown extension .cmap #7160

jomapp opened this issue Jul 3, 2024 · 1 comment · Fixed by #7312 or #7412
Labels
android issues that concern the android client but not all clients bug broken functionality, usability problems, unexpected errors ios issues that concern the ios client but not all clients state:tested We tested it and are about to release it
Milestone

Comments

@jomapp
Copy link
Contributor

jomapp commented Jul 3, 2024

Bug in mobile app

Describe the bug
When opening a PDF invoice on android (and probably also iOS), the app crashes as the ".cmap" extension is unknown.
".cmap" files are used to allow special font rendering inside the PDF invoices.

To Reproduce
Steps to reproduce the behavior:

  1. Go to 'payment'
  2. Click on 'show'
  3. Click on 'download'
  4. See that app crashes

Expected behavior
Downloading a PDF invoice is successful.

Smartphone (please complete the following information):

  • OS: Android 14
  • Version 234.20240627.0

Additional context

type: crash
osVersion: google/cheetah/cheetah:14/AP2A.240605.024/2024062700:user/release-keys
package: de.tutao.tutanota:396356
process: de.tutao.tutanota
processUptime: 14054 + 371 ms
installer: com.android.packageinstaller

java.lang.IllegalStateException: Unknown extension cmap for url https://assets.tutanota.com/pdf/identity_h.cmap
at de.tutao.tutanota.MainActivity.getMimeTypeForUrl(Unknown Source:142)
at de.tutao.tutanota.MainActivity.access$getMimeTypeForUrl(Unknown Source:0)
at de.tutao.tutanota.MainActivity$onCreate$2.shouldInterceptRequest(Unknown Source:216)
at WV.r5.a(chromium-TrichromeWebView6432.apk-stable-647812433:83)
at org.chromium.android_webview.AwContentsBackgroundThreadClient.shouldInterceptRequestFromNative(chromium-TrichromeWebView6432.apk-stable-647812433:15)
@jomapp jomapp added bug broken functionality, usability problems, unexpected errors android issues that concern the android client but not all clients ios issues that concern the ios client but not all clients labels Jul 3, 2024
@rezbyte rezbyte assigned rezbyte and unassigned rezbyte Jul 22, 2024
paw-hub added a commit that referenced this issue Jul 30, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
@paw-hub paw-hub self-assigned this Jul 30, 2024
@paw-hub paw-hub linked a pull request Jul 30, 2024 that will close this issue
paw-hub added a commit that referenced this issue Jul 30, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
charlag pushed a commit that referenced this issue Jul 30, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
paw-hub added a commit that referenced this issue Jul 30, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
paw-hub added a commit that referenced this issue Jul 30, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
paw-hub added a commit that referenced this issue Jul 30, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
paw-hub added a commit that referenced this issue Aug 2, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
rezbyte pushed a commit that referenced this issue Aug 6, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
@rezbyte rezbyte closed this as completed Aug 6, 2024
rezbyte pushed a commit that referenced this issue Aug 6, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
rezbyte pushed a commit that referenced this issue Aug 6, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
@rezbyte rezbyte added this to the 240.240807.0 milestone Aug 7, 2024
murilopereirame pushed a commit that referenced this issue Aug 8, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
@rezbyte rezbyte assigned rezbyte and unassigned rezbyte Aug 13, 2024
@rezbyte rezbyte added the state:tested-with-comments Issue has been tested but some issues were found and needs review. label Aug 13, 2024
@rezbyte
Copy link
Contributor

rezbyte commented Aug 13, 2024

See here.

domesticated-raptor pushed a commit that referenced this issue Aug 15, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
@charlag charlag reopened this Aug 16, 2024
@charlag charlag removed the state:tested-with-comments Issue has been tested but some issues were found and needs review. label Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
android issues that concern the android client but not all clients bug broken functionality, usability problems, unexpected errors ios issues that concern the ios client but not all clients state:tested We tested it and are about to release it
Projects
None yet
4 participants