Page MenuHomePhabricator

[native] Add base64 decode to CommUtilsModule
ClosedPublic

Authored by bartek on May 9 2023, 6:20 AM.
Tags
None
Referenced Files
F2334295: D7760.diff
Fri, Jul 26, 4:01 PM
Unknown Object (File)
Mon, Jul 15, 4:09 PM
Unknown Object (File)
Mon, Jul 15, 4:06 PM
Unknown Object (File)
Wed, Jul 10, 2:55 AM
Unknown Object (File)
Wed, Jul 10, 2:24 AM
Unknown Object (File)
Sat, Jul 6, 9:33 AM
Unknown Object (File)
Sat, Jul 6, 3:54 AM
Unknown Object (File)
Wed, Jul 3, 9:02 AM
Subscribers

Details

Summary

This adds JSI bindings for D7758 to the native CommUtilsModule.

Depends on D7758.

Test Plan

Encoded and decoded a Uint8Array into base64, using commCoreModule.base64{Encode,Decode}buffer() and console-log.

Used three different length of Uint8Arrays to test all padding values, e.g.:

new Uint8Array([1,2,3,4,5,6]);  // AQIDBAUG
new Uint8Array([1,2,3,4,5,6,7]); // AQIDBAUGBw==
new Uint8Array([1,2,3,4,5,6,7,8]); // AQIDBAUGBwg=

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bartek held this revision as a draft.
bartek published this revision for review.May 9 2023, 8:15 AM

Could you make sure in Test Plan that you tested decoding strings with each length of padding at the end (none, =, and ==)?

This revision is now accepted and ready to land.May 15 2023, 2:21 AM