iOS - Profile - Error message when uploading a photo using camera

[kavimuru]

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!

---

Action Performed:

1. Open staging New Expensify App
2. Log in
3. Select settings by clicking the avatar at the top right of the menu
4. Select Profile
5. Tap the camera icon on the avatar
6. From the pop up menu select Upload photo
7. Choose Take photo
8. Take a random image and upload

Expected Result:

Image would upload with no issues

Actual Result:

Error pop up each time and unable to upload image taken with error message "Image upload failed “The selected image exceeds the maximum upload size of 3MB”"
When you check the image size, it it actually less than 3MB

Workaround:

Unknown

Platform:

Where is this issue occurring?

• iOS

Version Number: 1.1.18-0

Reproducible in staging?: Yes

Reproducible in production?: Yes

Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Bug5362641_IMG_2016.mp4

Expensify/Expensify Issue URL:
Issue reported by: Applause
Slack conversation:

View all open jobs on GitHub

[MelvinBot]

Triggered auto assignment to @nickmurray47 (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

[nickmurray47]

Going to unassign for now since I'm running into issues with Xcode on dev. I'm assuming this can be worked on externally as long as there aren't any existing issues with our current file upload API in Web-E.

[akshayasalvi]

I had reported this earlier: #6554 and it was closed based on this #6554 (comment)

[MelvinBot]

Eep! 4 days overdue now. Issues have feelings too...

[MelvinBot]

6 days overdue. This is scarier than being forced to listen to Vogon poetry!

[MelvinBot]

10 days overdue. Is anyone even seeing these? Hello?

[Jag96]

It looks like we set the max value for the upload here, which was added in this PR. When testing, it looks like the values for taking a photo with the front camera and back camera were both over 3MB, here are those values (respectively):

Asked in slack here to confirm this value is ok to update to 6MB, otherwise we'll need to look into another fix for this. This can likely be external, but waiting for confirmation before updating.

[mananjadhav]

Proposal if made external

This is just a small config change. Need to update the value for CONST.AVATAR_MAX_ATTACHMENT_SIZE as 6291456.

I've checked that it successfully uploads and there isn't any error thrown from the API.

[Jag96]

Already had the branch up for this one so just made the PR myself