Commit dfd5b9af by Nick Rivadeneira Committed by Tute Costa

Update accepted server_side_encryption value (#2243)

AWS no longer accepts `:aes256` as an encryption value and will raise
`Aws::S3::Errors::InvalidArgument` with the message `The encryption method
specified is not supported`. It should instead be `"AES256"`.

This commit makes no implementation change, but for those that use
source code and tests as documentation, it should help prevent
confusion.
parent ee55bca3
...@@ -1207,7 +1207,7 @@ describe Paperclip::Storage::S3 do ...@@ -1207,7 +1207,7 @@ describe Paperclip::Storage::S3 do
'access_key_id' => "12345", 'access_key_id' => "12345",
'secret_access_key' => "54321" 'secret_access_key' => "54321"
}, },
s3_server_side_encryption: :aes256 s3_server_side_encryption: "AES256"
end end
context "when assigned" do context "when assigned" do
...@@ -1227,7 +1227,7 @@ describe Paperclip::Storage::S3 do ...@@ -1227,7 +1227,7 @@ describe Paperclip::Storage::S3 do
object.expects(:upload_file) object.expects(:upload_file)
.with(anything, content_type: "image/png", .with(anything, content_type: "image/png",
acl: :"public-read", acl: :"public-read",
server_side_encryption: :aes256) server_side_encryption: "AES256")
@dummy.save @dummy.save
end end
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment