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

Ability to provide server-side encryption for S3 access using ServerSideEncryption or ServerSideCustomerEncryption methods #332

Merged
merged 5 commits into from
Nov 27, 2019
Merged

Ability to provide server-side encryption for S3 access using ServerSideEncryption or ServerSideCustomerEncryption methods #332

merged 5 commits into from
Nov 27, 2019

Conversation

danielmarbach
Copy link
Contributor

Alternative for #315

@danielmarbach danielmarbach mentioned this pull request Nov 26, 2019
Closed
SeanFeldman
SeanFeldman previously approved these changes Nov 27, 2019
View reviewed changes
@danielmarbach danielmarbach changed the title WIP Ability to provide ServerSideEncryption or ServerSideCustomerEncryption for S3 access Ability to provide ServerSideEncryption or ServerSideCustomerEncryption for S3 access Nov 27, 2019
@danielmarbach danielmarbach added this to the 4.4.0 milestone Nov 27, 2019
@SeanFeldman SeanFeldman changed the title Ability to provide ServerSideEncryption or ServerSideCustomerEncryption for S3 access Ability to provide server-side encryption for S3 access using ServerSideEncryption or ServerSideCustomerEncryption methods Nov 27, 2019
SeanFeldman
SeanFeldman approved these changes Nov 27, 2019
View reviewed changes
Copy link
Contributor

@SeanFeldman SeanFeldman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good addition 👍

@danielmarbach danielmarbach merged commit 4015f03 into master Nov 27, 2019
@danielmarbach danielmarbach deleted the encryption branch November 27, 2019 20:42
@danielmarbach danielmarbach mentioned this pull request Nov 27, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SeanFeldman SeanFeldman SeanFeldman approved these changes

Assignees
No one assigned
Labels
Improvement
Projects
None yet
Milestone
4.4.0
Development

Successfully merging this pull request may close these issues.
2 participants
@danielmarbach @SeanFeldman