Skip to content

Conversation

@baloo
Copy link
Member

@baloo baloo commented Dec 9, 2025

The new API force the output to be a straight array, but there is a
custom drop implementation where we can put the Zeroize of the output.

@baloo
Copy link
Member Author

baloo commented Dec 9, 2025

upstream PR that introduces the generator API: rust-random/rand_core#26

I've tried to adjust the upstream API to see if I could bring back the support for Zeroize, but all I ended up with was essentially a straight revert of the API.

Copy link
Contributor

@dhardy dhardy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@baloo baloo marked this pull request as draft December 9, 2025 19:45
@baloo baloo force-pushed the baloo/push-sxqszypxlxnt branch from f57179d to 01043e2 Compare December 9, 2025 19:48
@baloo baloo marked this pull request as ready for review December 11, 2025 03:24
@baloo baloo force-pushed the baloo/push-sxqszypxlxnt branch from 01043e2 to 8dd5dfd Compare December 11, 2025 03:25
The new API force the output to be a straight array, but there is a
custom drop implementation where we can put the Zeroize of the output.
@baloo baloo force-pushed the baloo/push-sxqszypxlxnt branch from 8dd5dfd to 078bf89 Compare December 11, 2025 03:26
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.

3 participants