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.

Sign up for GitHub

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

Jump to bottom

7z checksums missing #5

Open
weltkante opened this issue Feb 24, 2015 · 2 comments
Open

7z checksums missing #5

weltkante opened this issue Feb 24, 2015 · 2 comments
Assignees
@weltkante
Labels
enhancement
Milestone
nuget-release

Comments

@weltkante
Copy link
Owner

While prototyping an AES encoder for issue #1 I noticed that the currently generated 7z archives don't include checksums of their files. That's ok because they are optional and for the previous use case (lzma support in duplicati) these weren't necessary because our callers were maintaining their own checksums. However for standalone 7z archives they are important to verify the integrity of the archive, in particular when streams are encrypted you need it to verify that the password was correct.
@weltkante weltkante self-assigned this Feb 24, 2015
@weltkante weltkante mentioned this issue Feb 24, 2015
Closed
@weltkante
Copy link
Owner Author

Stream checksums have been added, which is enough to detect corrupt archives or wrong encryption passwords. Checksums for individual files are still missing so I'll leave the issue open.

@weltkante
Copy link
Owner Author

This should be working, but I still need to verify this is working as intended before I can close this issue.

@weltkante weltkante added this to the nuget-release milestone Dec 29, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@weltkante weltkante

Labels
enhancement
Projects
None yet
Milestone
nuget-release
Development

No branches or pull requests
1 participant
@weltkante