This adds wrapper to olm method to sequential decrypt added in PR#11.
Details
Details
Tested in next diff in the stack
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
native/cpp/CommonCpp/CryptoTools/Session.cpp | ||
---|---|---|
160 ↗ | (On Diff #39368) | there is some code duplication but not sure if there is a cleaner way of deduplicating this |
native/cpp/CommonCpp/CryptoTools/Session.cpp | ||
---|---|---|
160 ↗ | (On Diff #39368) | Could we extract it to a common function that returns length for decrypted content buffer? This particular example seems to be too much duplication. |
202 ↗ | (On Diff #39368) | I think it wouldn't hurt to replace it with constructor that doesn't require casting: std::string{decryptedMessage.begin(), decryptedMessage.end()}; |