Skip to content

Conversation

@pdugre
Copy link
Contributor

@pdugre pdugre commented Dec 20, 2024

Currently crashing for memory corruption on the OnlineEncryptorLastChunk FFI function, I added a comment where.

I'll refer to @CBenoit for help to fix that part, as he knows everything there is to know about Rust.

@MathieuMorrissette
Copy link
Contributor

MathieuMorrissette commented Jan 14, 2025

TODO

  • Add decryption stream test ✅
  • Wasm test

@MathieuMorrissette MathieuMorrissette force-pushed the online-encryption branch 6 times, most recently from 366db61 to bd8a6b4 Compare January 15, 2025 15:01
@MathieuMorrissette
Copy link
Contributor

Wasm bindings will be implemented in another PR

@MathieuMorrissette MathieuMorrissette changed the title [NOT FINISHED] Online encryption Online encryption feature Jan 15, 2025
@MathieuMorrissette MathieuMorrissette marked this pull request as ready for review January 15, 2025 16:10
@MathieuMorrissette MathieuMorrissette requested a review from a team as a code owner January 15, 2025 16:10
@MathieuMorrissette MathieuMorrissette merged commit dafef3b into master Jan 15, 2025
25 checks passed
DannyBedard pushed a commit that referenced this pull request Jan 21, 2025
* [STREAM] WIP commit

* [STREAM] WIP commit

* [STREAM] Initial V1 implementation

* [STREAM] Generic bindings

* [STREAM] Fix warnings

* [STREAM] Remove useless info in encryptor/decryptor structs

* [STREAM] Write macro to remove code duplication

* [STREAM] WIP make use of generics

* [STREAM] temporarly fix the trait system with a downcast

* [STREAM] revert symmetric/asymmetric trait to be dynamic

* [STREAM] FFI bindings

* [STREAM] C# EncryptionStream

* [STREAM] Attempt to fix segfault

* [WIP] Fix last chunk size

* [WIP] Cargo fmt

* dotnet format

* Decrypt Stream Test C#

* test

* changelog

* fix nuget

* update crates

---------

Co-authored-by: Mathieu Morrissette <mathieumorrissette@gmail.com>
@MathieuMorrissette MathieuMorrissette deleted the online-encryption branch February 12, 2025 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants