Skip to content

Commit

Permalink
Remove ASSET_HOST env var
Browse files Browse the repository at this point in the history
This env var is no longer in active use since the introduction of Static
proxy [1] and usage of it will soon have problems with the Content
Security Policy [2].

[1]: https://github.com/alphagov/govuk_app_config/blob/7f060692720df50a27f6845f052b04eae2246226/lib/govuk_app_config/govuk_proxy/static_proxy.rb
[2]: alphagov/govuk_app_config#274
  • Loading branch information
kevindew committed Dec 29, 2022
1 parent 54d0d84 commit 35ecec7
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions config/application.rb
Original file line number Diff line number Diff line change
Expand Up @@ -125,10 +125,6 @@ class Application < Rails::Application
# https://github.com/alphagov/govuk-frontend/issues/1350
config.assets.css_compressor = nil

# allow overriding the asset host with an enironment variable, useful for
# when router is proxying to this app but asset proxying isn't set up.
config.asset_host = ENV["ASSET_HOST"]

# Do not swallow errors in after_commit/after_rollback callbacks.
# config.active_record.raise_in_transactional_callbacks = true
end
Expand Down

0 comments on commit 35ecec7

Please sign in to comment.