From c4b34791733de5b396a726aeff3f16d349e56c7e Mon Sep 17 00:00:00 2001 From: Thibaut Girka Date: Tue, 28 Aug 2018 21:41:03 +0200 Subject: [PATCH] Fix CSP with S3/SWIFT hosts --- config/initializers/content_security_policy.rb | 15 +++++++++++++-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/config/initializers/content_security_policy.rb b/config/initializers/content_security_policy.rb index 1308f0fd1d..6d7666c48e 100644 --- a/config/initializers/content_security_policy.rb +++ b/config/initializers/content_security_policy.rb @@ -4,6 +4,16 @@ if Rails.env.production? assets_host = Rails.configuration.action_controller.asset_host || "https://#{ENV['WEB_DOMAIN'] || ENV['LOCAL_DOMAIN']}" + data_hosts = [assets_host] + + if ENV['S3_ENABLED'] == 'true' + attachments_host = ENV['S3_ALIAS_HOST'] || ENV['S3_CLOUDFRONT_HOST'] || ENV['S3_HOSTNAME'] || "s3-#{ENV['S3_REGION'] || 'us-east-1'}.amazonaws.com" + elsif ENV['SWIFT_ENABLED'] == 'true' + attachments_host = ENV['SWIFT_OBJECT_URL'] + else + attachments_host = nil + end + data_hosts << attachments_host unless attachments_host.nil? Rails.application.config.content_security_policy do |p| p.base_uri :none @@ -13,9 +23,10 @@ if Rails.env.production? p.font_src :self, assets_host p.img_src :self, :https, :data, :blob p.style_src :self, :unsafe_inline, assets_host - p.media_src :self, :data, assets_host + p.media_src :self, :data, *data_hosts p.frame_src :self, :https - p.connect_src :self, :blob, assets_host, Rails.configuration.x.streaming_api_base_url + p.worker_src :self, assets_host + p.connect_src :self, :blob, Rails.configuration.x.streaming_api_base_url, *data_hosts end end