HomePhabricator
Diffusion Comm cf4e8423f558

[native] Fix encrypted media upload failures

Description

[native] Fix encrypted media upload failures

Summary:
Fixes ENG-4566. Issue details are in the task.

Previously, temporary encrypted file was created in the same directory as original. This diffs moves it to the app cache dir.

Test Plan:
Ensured that encrypted media are successfully uploaded on both iOS (physical device) and Android. Also ensured that it is properly disposed after upload.

Console logged modified paths to confirm that they're correctly costructed in the app cache dir:

{
  "destinationPath": "/Users/barthap/Library/Developer/CoreSimulator/Devices/4BFD741A-64AB-4720-8125-3874478D8B4B/data/Containers/Data/Application/64F264B3-A8CF-4237-9B72-A2EFCE8B8A73/tmp/IMG_0011.dat",
  "destinationURI":"file:///Users/barthap/Library/Developer/CoreSimulator/Devices/4BFD741A-64AB-4720-8125-3874478D8B4B/data/Containers/Data/Application/64F264B3-A8CF-4237-9B72-A2EFCE8B8A73/tmp/IMG_0011.dat",
  "originalFilename":"IMG_0011.PNG",
  "targetFilename":"IMG_0011.dat",
  "uri":"file:///Users/barthap/Library/Developer/CoreSimulator/Devices/4BFD741A-64AB-4720-8125-3874478D8B4B/data/Media/DCIM/100APPLE/IMG_0011.PNG"
}

Reviewers: atul, ashoat, ginsu

Reviewed By: ashoat

Subscribers: tomek

Differential Revision: https://phab.comm.dev/D8738

Details

Provenance
bartekAuthored on Aug 5 2023, 9:22 AM
Reviewer
ashoat
Differential Revision
D8738: [native] Fix encrypted media upload failures
Parents
rCOMM89e87a98637a: [native] Fix missing media mission report steps
Branches
Unknown
Tags
Unknown