Skip to content

aead: add alloc-based APIs to stream module#447

Merged
tarcieri merged 1 commit intomasterfrom
aead/stream-alloc-support
Jan 5, 2021
Merged

aead: add alloc-based APIs to stream module#447
tarcieri merged 1 commit intomasterfrom
aead/stream-alloc-support

Conversation

@tarcieri
Copy link
Member

@tarcieri tarcieri commented Jan 4, 2021

Adds a corresponding set of allocating APIs to StreamPrimitive, Decryptor, and Encryptor.

@tarcieri tarcieri requested review from newpavlov and str4d January 4, 2021 20:31
@tarcieri
Copy link
Member Author

tarcieri commented Jan 4, 2021

This PR contains a bit more copy-paste than I'd like, which generally speaks to some composability problems with the current trait design (both in the stream module and the aead module in general).

That said, I'd like to at least spike out the functionality, and then pursue addressing the above by exploring some breaking changes to the aead crate in general.

@tarcieri tarcieri force-pushed the aead/stream-alloc-support branch from f63b187 to 2856751 Compare January 4, 2021 20:33
Adds a corresponding set of allocating APIs to `StreamPrimitive`,
`Decryptor`, and `Encryptor`.
@tarcieri tarcieri force-pushed the aead/stream-alloc-support branch from 2856751 to 3c28547 Compare January 4, 2021 20:34
@tarcieri
Copy link
Member Author

tarcieri commented Jan 5, 2021

Merging this, but will post a follow-up issue on some potential (breaking) ways to refactor and clean up the aead crate.

@tarcieri tarcieri merged commit 20f6d24 into master Jan 5, 2021
@tarcieri tarcieri deleted the aead/stream-alloc-support branch January 5, 2021 14:49
@tarcieri tarcieri mentioned this pull request Feb 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant