[$250 Reproducible Steps][$250] Android - Attachment - The app crashes upon uploading a large PDF

[IuliiaHerets]

Deliverable

The deliverable is to provide reliable reproduction steps for the bug. - Follow the Propose a solution for a job process to submit the steps for review”. Inc. preconditions and additional details if they’re helpful. (ie. require x beta to be enabled, must be on a Collect plan)

---

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!

---

Version Number: v9.0.24-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4859370&group_by=cases:section_id&group_id=292107&group_order=asc
Issue reported by: Applause Internal Team

Action Performed:

1. Open the app
2. Log in with any Gmail account
3. Open any 1:1 chat or self DM
4. Navigate to the Composers "+" button - Add attachment
5. Tap on "Choose file"
6. Upload the example PDF attached to this issue

Expected Result:

I should be able to upload any PDF attachment.

Actual Result:

The app crashes upon uploading a large PDF (20 MB). I'm able to upload smaller PDF attachments.

Workaround:

Unknown

Platforms:

• Android: Native
• Android: mWeb Chrome
• iOS: Native
• iOS: mWeb Safari
• MacOS: Chrome / Safari
• MacOS: Desktop

Screenshots/Videos

2308.txt

Bug6572320_1724431215191.az_recorder_20240823_183839.mp4

Bug6572320_1723663373443!English_Vocabulary_in_Use_Elementary_2017.pdf

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~01b178b0b236b5c693
• Upwork Job ID: 1828189418619467640
• Last Price Increase: 2024-11-04

[melvin-bot]

Triggered auto assignment to @johncschuster (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

[IuliiaHerets]

@johncschuster FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

[melvin-bot]

@johncschuster Whoops! This issue is 2 days overdue. Let's get this updated quick!

[melvin-bot]

Job added to Upwork: https://www.upwork.com/jobs/~01b178b0b236b5c693

[melvin-bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @mkhutornyi (External)

[rezkiy37]

Hi, I am Michael (Mykhailo) from Callstack, an expert agency and I can work on this issue.

[rezkiy37]

The issue is not replicable on the latest Android production. I am trying to test it in development right now.

New.Expensify.mp4

[rezkiy37]

The bug is not replicable on the latest main. So @IuliiaHerets @mkhutornyi please confirm if you can reproduce it.

New.Expensify.Dev.mp4

[mkhutornyi]

Not able to reproduce crash but I am seeing file attachment instead of pdf preview.

[IuliiaHerets]

@rezkiy37 Tester still can reproduce a crash, build - 9.0.26-1

az_recorder_20240829_064541.mp4

[rezkiy37]

Not able to reproduce crash but I am seeing file attachment instead of pdf preview.

I don't think it is a bug. I believe it is expected behavior.

[rezkiy37]

I could reproduce the bug in the dev build; the version is 9.0.26-1. The file size does not matter to me. It crashes with small files 2 pages 30.66 KB (small.pdf). @IuliiaHerets can you please try one more time with small files as well?

Details

Bug.mp4

[IuliiaHerets]

@rezkiy37 We unable to repro with small PDF attachments

az_recorder_20240829_192650.mp4

[rezkiy37]

Looks like I've found the root cause of the bug. It happens because of this line:

App/src/pages/home/report/ReportActionCompose/ReportActionCompose.tsx

Line 292 in 1f71ce6

runOnUI(clear)();

The UI thread cannot execute some JS logic. I think it is because of the version mismatch of react-native and react-native-reanimated. I am trying to fix it.

[rezkiy37]

@IuliiaHerets, please clear the app's cache and reinstall it to reproduce the bug again. I've tried on many production versions and I don't have the bug. However, the bug is replicable in development.

Details

[melvin-bot]

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

[IuliiaHerets]

@rezkiy37 Tester still can repro the issue in 9.0.28-0 after a reinstall

az_recorder_20240903_131956.mp4

[rezkiy37]

Hey!
I will be OOO from 04.09 (afternoon) till 09.09. I will continue to work on this one once I am back.

[melvin-bot]

@johncschuster, @mkhutornyi Eep! 4 days overdue now. Issues have feelings too...

[johncschuster]

Following the SO above:

If no one is able to provide steps after a week or two, and the BZ thinks the bug is important to fix, BZ should raise the price to $250

If no one is able to reproduce and no proposals have been submitted after 4 weeks, close the issue.

[melvin-bot]

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

[johncschuster]

Bumping while we wait on reproducible steps

[melvin-bot]

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

[mvtglobally]

Issue not reproducible during KI retests. (First week)

[melvin-bot]

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

[johncschuster]

Let's get one more retest and then close it.

[melvin-bot]

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

[mvtglobally]

Issue not reproducible during KI retests. (Second week)

[mkhutornyi]

Let's close then

[rezkiy37]

Agree

[melvin-bot]

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

[rezkiy37]

Let's get one more retest and then close it.

Issue not reproducible during KI retests. (Second week)

@johncschuster, I think we can close it now.

[mvtglobally]

Issue not reproducible during KI retests. (Third week)