-
Notifications
You must be signed in to change notification settings - Fork 147
Backup & Restore: support for SSE-C #1520
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
Backup & Restore: support for SSE-C #1520
Conversation
|
Thanks for this contribution @xavierleune, we are willing to support this. I'm on holiday currently, but @Michaelpalacce might be able to do an initial review |
102d8cf to
a18adf3
Compare
|
Thanks @mmontes11, enjoy your holiday ! |
a18adf3 to
946a704
Compare
mmontes11
left a comment
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.
Left some comments after an initial review. This still needs to be validated with our development Minio installation.
mmontes11
left a comment
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.
Managed to validate this in our local MinIO installation. Left some additional comments
mmontes11
left a comment
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.
LGTM! Thanks for this contribution @xavierleune ! I'll make the adaptations of the docs myself
|
Thanks @mmontes11 |
This pull requests adds support for SSE-C, thanks to minio-go that already supports it.
Fixes #1084