I have introduced changes into the S3FileSystemProvider and S3OutputStream classes to support server-side encryption.
In particular:
A new method setStorageEncryption has been created in the S3OutputStream class to read the configuration options and, if specified, apply AES256 encryption to every new file that is uploaded on S3.
Tests are also provided, please consider that some previous tests for FileSystemProvider#copy method are returning an error, since they use mock-ups for S3 connection and the new method requires real objects on S3 to retrieve the meta-data information (maybe those tests could be suppressed or modified). Other tests in the suite also seem to fail, but at least for what I have been able to check, these errors are not related to the changes in this pull request.
I have introduced changes into the S3FileSystemProvider and S3OutputStream classes to support server-side encryption.
In particular:
A new method
setStorageEncryptionhas been created in the S3OutputStream class to read the configuration options and, if specified, apply AES256 encryption to every new file that is uploaded on S3.The
copymethod in the S3FileSystemProvider class has been updated to use CopyObjectRequest to define encryption for the target object, if the source object is encrypted. By default, the AWS JDK method copyObject does not transfer encryption information (http://docs.aws.amazon.com/AWSJavaSDK/latest/javadoc/com/amazonaws/services/s3/AmazonS3Client.html#copyObject-com.amazonaws.services.s3.model.CopyObjectRequest-) so apparently this needs to be managed explicitly.Tests are also provided, please consider that some previous tests for FileSystemProvider#copy method are returning an error, since they use mock-ups for S3 connection and the new method requires real objects on S3 to retrieve the meta-data information (maybe those tests could be suppressed or modified). Other tests in the suite also seem to fail, but at least for what I have been able to check, these errors are not related to the changes in this pull request.