From 23601914345d201b8f06ceea637c70e0a23674db Mon Sep 17 00:00:00 2001 From: Takeshi Umeda Date: Fri, 23 Apr 2021 03:33:36 +0900 Subject: [PATCH] Fix guard against DNS rebinding attacks (#16095) --- config/initializers/1_hosts.rb | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/config/initializers/1_hosts.rb b/config/initializers/1_hosts.rb index 0ce4320b7..eaf6e0181 100644 --- a/config/initializers/1_hosts.rb +++ b/config/initializers/1_hosts.rb @@ -4,7 +4,7 @@ port = ENV.fetch('PORT') { 3000 } host = ENV.fetch('LOCAL_DOMAIN') { "localhost:#{port}" } web_host = ENV.fetch('WEB_DOMAIN') { host } -alternate_domains = ENV.fetch('ALTERNATE_DOMAINS') { '' } +alternate_domains = ENV.fetch('ALTERNATE_DOMAINS') { '' }.split(/\s*,\s*/) Rails.application.configure do https = Rails.env.production? || ENV['LOCAL_HTTPS'] == 'true' @@ -15,7 +15,7 @@ Rails.application.configure do config.x.use_s3 = ENV['S3_ENABLED'] == 'true' config.x.use_swift = ENV['SWIFT_ENABLED'] == 'true' - config.x.alternate_domains = alternate_domains.split(/\s*,\s*/) + config.x.alternate_domains = alternate_domains config.action_mailer.default_url_options = { host: web_host, protocol: https ? 'https://' : 'http://', trailing_slash: false } @@ -30,6 +30,6 @@ Rails.application.configure do unless Rails.env.test? config.hosts << host if host.present? config.hosts << web_host if web_host.present? - config.hosts << alternate_domains if alternate_domains.present? + config.hosts.concat(alternate_domains) if alternate_domains.present? end end