Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: support crc32c checksum validation #173

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

tam7t
Copy link
Contributor

@tam7t tam7t commented May 26, 2022

SecretVersions created after July 2021 all have a checksum either
generated by the end user when calling AddSecretVersion or server
side by the Secret Manager API when initially written.

https://cloud.google.com/secret-manager/docs/data-integrity

This change adds an additional crc32c validation to catch possible
data corruption. Note this will not catch corruption that may occur
downstream of the provider (i.e. by the driver process, linux tmpfs
mount, or the process consuming the secret).

SecretVersions created after July 2021 all have a checksum either
generated by the end user when calling AddSecretVersion or server
side by the Secret Manager API when initially written.

https://cloud.google.com/secret-manager/docs/data-integrity

This change adds an additional crc32c validation to catch possible
data corruption. Note this will not catch corruption that may occur
downstream of the provider (i.e. by the driver process, linux tmpfs
mount, or the process consuming the secret).
@tam7t tam7t temporarily deployed to e2e-test May 27, 2022 00:23 Inactive
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