diff options
author | David Heinemeier Hansson <david@loudthinking.com> | 2017-07-23 11:06:06 -0500 |
---|---|---|
committer | David Heinemeier Hansson <david@loudthinking.com> | 2017-07-23 11:06:06 -0500 |
commit | c285c6824dc186e00040b7283877fea917050275 (patch) | |
tree | a476b09a5702796e18fe4ec753919afae6832d79 | |
parent | 46da4ee7daf1ecaa2fc47a260ccb58e119a1b5ea (diff) | |
download | rails-c285c6824dc186e00040b7283877fea917050275.tar.gz rails-c285c6824dc186e00040b7283877fea917050275.tar.bz2 rails-c285c6824dc186e00040b7283877fea917050275.zip |
Provide a BlobsController for stable blob URLs
We need to have stable urls for blobs and variants or caching won't work. So provide a controller that can give that and redirect to the service URL upon lookup.
-rw-r--r-- | app/controllers/active_storage/blobs_controller.rb | 22 | ||||
-rw-r--r-- | config/routes.rb | 8 |
2 files changed, 30 insertions, 0 deletions
diff --git a/app/controllers/active_storage/blobs_controller.rb b/app/controllers/active_storage/blobs_controller.rb new file mode 100644 index 0000000000..5a527d0a33 --- /dev/null +++ b/app/controllers/active_storage/blobs_controller.rb @@ -0,0 +1,22 @@ +# Take a signed permanent reference for a blob and turn it into an expiring service URL for its download. +# Note: These URLs are publicly accessible. If you need to enforce access protection beyond the +# security-through-obscurity factor of the signed blob references, you'll need to implement your own +# authenticated redirection controller. +class ActiveStorage::BlobsController < ActionController::Base + def show + if blob = find_signed_blob + redirect_to blob.url(disposition: disposition_param) + else + head :not_found + end + end + + private + def find_signed_blob + ActiveStorage::Blob.find_signed(params[:signed_id]) + end + + def disposition_param + params[:disposition].presence_in(%w( inline attachment )) || "inline" + end +end diff --git a/config/routes.rb b/config/routes.rb index 78fa0e707b..b368e35cac 100644 --- a/config/routes.rb +++ b/config/routes.rb @@ -1,4 +1,12 @@ Rails.application.routes.draw do + get "/rails/active_storage/blobs/:signed_id/*filename" => "active_storage/blobs#show", as: :rails_service_blob + + direct :rails_blob do |blob| + route_for(:rails_service_blob, blob.signed_id, blob.filename) + end + + resolve("ActiveStorage::Blob") { |blob| route_for(:rails_blob, blob) } + resolve("ActiveStorage::Attachment") { |attachment| route_for(:rails_blob, attachment.blob) } get "/rails/active_storage/variants/:signed_blob_id/:variation_key/*filename" => "active_storage/variants#show", as: :rails_blob_variation |