[native] Fix sending videos on iOS 18
ClosedPublic
Actions

Authored by angelika on Oct 24 2024, 6:37 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Dec 16, 8:44 PM

	Unknown Object (File)
	Mon, Dec 16, 8:44 PM

	Unknown Object (File)
	Mon, Dec 16, 8:44 PM

	Unknown Object (File)
	Mon, Dec 16, 8:44 PM

	Unknown Object (File)
	Wed, Dec 4, 10:31 AM

	Unknown Object (File)
	Wed, Dec 4, 10:31 AM

	Unknown Object (File)
	Wed, Dec 4, 10:31 AM

	Unknown Object (File)
	Wed, Dec 4, 10:08 AM

View All 28 Files

Subscribers

ashoat

Details

Reviewers

tomek
kamil
ashoat

Commits

rCOMMfad6010b41b8: [native] Fix sending videos on iOS 18

Summary

On iOS 18 getAssetInfoAsync returns uri with a weird #YnBsaXN... suffix. This looks like iOS bug so for now to mitigate it we simply remove the suffix.

Test Plan

Send video on iOS 18 and 15.

Diff Detail

Repository

rCOMM Comm

Branch

graszka22/ENG-9736

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

angelika created this revision.Oct 24 2024, 6:37 AM

Herald added a subscriber: ashoat. · View Herald TranscriptOct 24 2024, 6:37 AM

ashoat published this revision for review.Oct 24 2024, 7:03 AM

ashoat accepted this revision.

This revision is now accepted and ready to land.Oct 24 2024, 7:03 AM

Harbormaster completed remote builds in B32366: Diff 45374.Oct 24 2024, 7:16 AM

Closed by commit rCOMMfad6010b41b8: [native] Fix sending videos on iOS 18 (authored by angelika). · Explain WhyOct 24 2024, 12:26 PM

This revision was automatically updated to reflect the committed changes.

angelika added a commit: rCOMMfad6010b41b8: [native] Fix sending videos on iOS 18.

Revision Contents
Changeset List

Path

Size

native/

media/

file-utils.js

7 lines

Diff 45374

View Options

native/media/file-utils.js

[native] Fix sending videos on iOS 18ClosedPublicActions