Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Rails default config for 8.0 #1588

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 14 additions & 1 deletion config/application.rb
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
module AssetManager
class Application < Rails::Application
# Initialize configuration defaults for originally generated Rails version.
config.load_defaults 7.2
config.load_defaults 8.0

# Please, add to the `ignore` list any other `lib` subdirectories that do
# not contain `.rb` files, or that should not be reloaded or eager loaded.
Expand All @@ -47,6 +47,19 @@ class Application < Rails::Application
# Disable Rack::Cache
config.action_dispatch.rack_cache = nil

# ActionDispatch strict freshness
#
# Configures whether the ActionDispatch::ETag middleware should prefer the
# ETag header over the Last-Modified header when both are present in the
# response.
#
# If set to true, when both headers are present only the ETag is considered
# as specified by RFC 7232 section 6.
#
# If set to false, when both headers are present, both headers are checked
# and both need to match for the response to be considered fresh.
config.action_dispatch.strict_freshness = false

config.assets.prefix = "/asset-manager"

unless Rails.application.config_for(:secrets).jwt_auth_secret
Expand Down
Loading