Skip to content

Commit 177fc82

Browse files
prathamesh-sonpatkirafaelfranca
authored andcommitted
Merge pull request rails#25720 from timrogers/actiondispatch-ssl-defaults-docs
Correct defaults in documentation for ActionDispatch::SSL
1 parent 69753f4 commit 177fc82

File tree

1 file changed

+5
-4
lines changed
  • actionpack/lib/action_dispatch/middleware

1 file changed

+5
-4
lines changed

actionpack/lib/action_dispatch/middleware/ssl.rb

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,17 +18,18 @@ module ActionDispatch
1818
# Enabled by default. Configure `config.ssl_options` with `hsts: false` to disable.
1919
#
2020
# Set `config.ssl_options` with `hsts: { … }` to configure HSTS:
21-
# * `expires`: How long, in seconds, these settings will stick. Defaults to
22-
# `180.days` (recommended). The minimum required to qualify for browser
23-
# preload lists is `18.weeks`.
21+
# * `expires`: How long, in seconds, these settings will stick. The minimum
22+
# required to qualify for browser preload lists is `18.weeks`. Defaults to
23+
# `180.days` (recommended).
2424
# * `subdomains`: Set to `true` to tell the browser to apply these settings
2525
# to all subdomains. This protects your cookies from interception by a
26-
# vulnerable site on a subdomain. Defaults to `true`.
26+
# vulnerable site on a subdomain. Defaults to `false`.
2727
# * `preload`: Advertise that this site may be included in browsers'
2828
# preloaded HSTS lists. HSTS protects your site on every visit *except the
2929
# first visit* since it hasn't seen your HSTS header yet. To close this
3030
# gap, browser vendors include a baked-in list of HSTS-enabled sites.
3131
# Go to https://hstspreload.appspot.com to submit your site for inclusion.
32+
# Defaults to `false`.
3233
#
3334
# To turn off HSTS, omitting the header is not enough. Browsers will remember the
3435
# original HSTS directive until it expires. Instead, use the header to tell browsers to

0 commit comments

Comments
 (0)