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

Support optional checksum for reading & writing files #177

Open
chipsenkbeil opened this issue May 8, 2023 · 0 comments
Open

Support optional checksum for reading & writing files #177

chipsenkbeil opened this issue May 8, 2023 · 0 comments
Labels
enhancement New feature or request
Milestone

Comments

@chipsenkbeil
Copy link
Owner

A feature that would be handy is to support an optional field that is a checksum. This would be used in two ways

  1. When reading a file's contents (bytes or text), we could include a field that says include_checksum. If true, then the server would compute a checksum of the file to send back that can be used to verify if the content has changed. Alternatively, we make this part of the metadata? Or have another command to do so. Not sure.
  2. When writing a file's contents (bytes or text), we could include a field that says checksum. This would be the checksum of the file's contents presently. If provided and the file's checksum does not match, then the write would fail as the file had changed since the last attempt. Would be handy to ensure that we don't overwrite a remote file.

Also, we'd need to figure out if we want to specify a checksum type or just support a singular type like sha256.

@chipsenkbeil chipsenkbeil added the enhancement New feature or request label May 8, 2023
@chipsenkbeil chipsenkbeil added this to the 0.20 milestone May 8, 2023
@chipsenkbeil chipsenkbeil modified the milestones: 0.20, 1.0 Jun 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant