-
Notifications
You must be signed in to change notification settings - Fork 16
feat(cfb-mode): add Buffer{Encryptor|Decryptor} #17
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
Conversation
This allows encrypting and decrypting based on multiple individual pieces of the data, rather than having to work with all data at once.
ping @tarcieri @newpavlov |
I'm personally fine with this, although I'm curious what @newpavlov has to say |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the late review! I have some minor nits/suggestions and the code can be improved in regards of eliminating unreachable panics, but otherwise I think it's good to merge. The comments about decryptor symmetrically apply to the encryptor as well. I will try to do the panics stuff later in a separate PR after releasing these changes.
@newpavlov should be ready for rereview |
Ping @dignifiedquire @newpavlov 2 months passed since last activity here, just want to make sure this PR is not forgotten and bitrot along with rpgp/rpgp#165 |
Oh, I indeed forgot about it during my break from GitHub. I will try to refresh my memory about this PR today or tomorrow and assuming there are no issues (which is quite likely), I will merge and release it right away. |
This allows encrypting and decrypting based on multiple individual pieces of the data, rather than having to work with all data at once.