Page MenuHomePhabricator

[CommCoreModule] implement decrypting content with `olm` session
ClosedPublic

Authored by kamil on Dec 18 2023, 4:42 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Oct 19, 3:21 PM
Unknown Object (File)
Fri, Oct 18, 2:08 AM
Unknown Object (File)
Fri, Oct 18, 2:08 AM
Unknown Object (File)
Fri, Oct 18, 2:08 AM
Unknown Object (File)
Fri, Oct 18, 2:08 AM
Unknown Object (File)
Fri, Oct 18, 2:08 AM
Unknown Object (File)
Fri, Oct 18, 2:08 AM
Unknown Object (File)
Fri, Oct 18, 2:08 AM
Subscribers

Details

Summary

Implemented method to decrypt content.

Depends on D10377

Test Plan

This code is really hard to test as it's impossible to verify partial results.
I generated some keys and values, verified them with olm-utils.test.js, and checked if they succeeded/failed when calling the method introduced here in the same cases as JS tests.
Full functionality tested later in the stack.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil held this revision as a draft.
kamil published this revision for review.Dec 18 2023, 6:52 AM
native/cpp/CommonCpp/NativeModules/CommCoreModule.cpp
869 ↗(On Diff #34792)
marcin requested changes to this revision.Dec 21 2023, 3:24 AM
This revision now requires changes to proceed.Dec 21 2023, 3:24 AM
marcin added inline comments.
native/cpp/CommonCpp/NativeModules/CommCoreModule.cpp
859 ↗(On Diff #35144)

This file contains definition of EncryptedData so perhaps we could move this constant there?

This revision is now accepted and ready to land.Jan 2 2024, 11:59 PM

move definition to Tools.h

CI failure not related - landing

This revision was landed with ongoing or failed builds.Jan 8 2024, 6:48 AM
This revision was automatically updated to reflect the committed changes.