TEST_CODEOWNERS/rails
3
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Add content_type to upload in Azure

Browse Source
This commit is contained in:
Peter Zhu 2019-07-19 10:41:38 -04:00
parent e4747a4965
commit 094fa9277d
2 changed files with 16 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
activestorage/lib/active_storage/service/azure_storage_service.rb
View File

@ -17,10 +17,10 @@ def initialize(storage_account_name:, storage_access_key:, container:, **options
@container = container
end
def upload(key, io, checksum: nil, **)
def upload(key, io, checksum: nil, content_type: nil, **)
instrument :upload, key: key, checksum: checksum do
handle_errors do
blobs.create_block_blob(container, key, IO.try_convert(io) || io, content_md5: checksum)
blobs.create_block_blob(container, key, IO.try_convert(io) || io, content_md5: checksum, content_type: content_type)
end
end
end

14
activestorage/test/service/azure_storage_service_test.rb
View File

@ -9,6 +9,20 @@ class ActiveStorage::Service::AzureStorageServiceTest < ActiveSupport::TestCase
include ActiveStorage::Service::SharedServiceTests
test "upload with content_type" do
key = SecureRandom.base58(24)
data = "Foobar"
@service.upload(key, StringIO.new(data), checksum: Digest::MD5.base64digest(data), filename: ActiveStorage::Filename.new("test.txt"), content_type: "text/plain")
url = @service.url(key, expires_in: 2.minutes, disposition: :attachment, content_type: nil, filename: ActiveStorage::Filename.new("test.html"))
response = Net::HTTP.get_response(URI(url))
assert_equal "text/plain", response.content_type
assert_match(/attachment;.*test\.html/, response["Content-Disposition"])
ensure
@service.delete key
end
test "signed URL generation" do
url = @service.url(@key, expires_in: 5.minutes,
disposition: :inline, filename: ActiveStorage::Filename.new("avatar.png"), content_type: "image/png")