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.

Sign up for GitHub

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

Jump to bottom

Update production.rb template to provide example to exclude healthcheck from force_ssl redirect #51680

Merged
merged 1 commit into from Apr 28, 2024
Merged

Update production.rb template to provide example to exclude healthcheck from force_ssl redirect #51680

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 2 additions & 0 deletions railties/lib/rails/generators/rails/app/templates/config/environments/production.rb.tt
View file
Open in desktop
Expand Up @@ -58,6 +58,8 @@ Rails.application.configure do

# Force all access to the app over SSL, use Strict-Transport-Security, and use secure cookies.
config.force_ssl = true
# Skip http-to-https redirect for the default health check endpoint.
# config.ssl_options = { redirect: { exclude: ->(request) { request.path == "/up" } } }
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would Rails.application.routes.url_helpers.rails_health_check_path make sense here?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it would force evaluating the routes earlier. So not great.


# Log to STDOUT by default
config.logger = ActiveSupport::Logger.new(STDOUT)
Expand Down