Skip to content

md5: remove zeroize - #100

Merged
ahl merged 1 commit into
mainfrom
trey/md5-no-zero
Sep 2, 2026
Merged

md5: remove zeroize#100
ahl merged 1 commit into
mainfrom
trey/md5-no-zero

Conversation

@taspelund

Copy link
Copy Markdown
Contributor

There currently isn't a strong use case for zeroize with regards to this type, and it is preferable to remove zeroize instead of giving a false sense of security around the handling of the inner data. This can be restored or implemented in a different way if/when a clear security posture arrives.

There currently isn't a strong use case for zeroize with regards to this
type, and it is preferable to remove zeroize instead of giving a false
sense of security around the handling of the inner data. This can be
restored or implemented in a different way if/when a clear security
posture arrives.

Signed-off-by: Trey Aspelund <trey@oxidecomputer.com>
@taspelund taspelund self-assigned this Sep 2, 2026
@taspelund taspelund added the rust Pull requests that update rust code label Sep 2, 2026

@ahl ahl left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

thanks for going back and doing this

@ahl
ahl merged commit 30aad33 into main Sep 2, 2026
14 checks passed
@ahl
ahl deleted the trey/md5-no-zero branch September 2, 2026 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rust Pull requests that update rust code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants