Skip to content

Sync config/initializers/devise.rb with devise 4.9.3 template#301

Merged
tom93 merged 1 commit intomasterfrom
update-devise-config
May 21, 2025
Merged

Sync config/initializers/devise.rb with devise 4.9.3 template#301
tom93 merged 1 commit intomasterfrom
update-devise-config

Conversation

@tom93
Copy link
Contributor

@tom93 tom93 commented May 21, 2025

Changes:

  • config.stretches is now 1 in the test env, which makes the tests much faster (see #290).

  • config.stretches increased from 10 to 12 in non-test env (note that the cost increases exponentially).

  • Adopt new (more permissive) defaults for config.maximum_attempts and config.reset_password_within.

  • The template now sets config.skip_session_storage = [:http_auth], but it shouldn't affect us because we appear to use :param_auth (checked by logging the value during login).

  • The template now sets expire_all_remember_me_on_sign_out = true, but this is the default so it doesn't make a difference.

  • Some defaults commented out.

Changes:

 - config.stretches is now 1 in the test env, which makes the tests
   faster.

 - config.stretches increased from 10 to 12 in non-test env (note that
   the cost increases exponentially).

 - Adopt new (more permissive) defaults for config.maximum_attempts
   and config.reset_password_within.

 - The template now sets `config.skip_session_storage = [:http_auth]`,
   but it shouldn't affect us because we appear to use :param_auth
   (checked by logging the value during login).

 - The template now sets `expire_all_remember_me_on_sign_out = true`,
   but this is the default so it doesn't make a difference.

 - Some defaults commented out.
@coveralls
Copy link

Coverage Status

coverage: 37.513%. remained the same
when pulling 88acbb0 on update-devise-config
into 1b0fd46 on master.

@tom93 tom93 merged commit a036d7a into master May 21, 2025
6 checks passed
@tom93 tom93 deleted the update-devise-config branch May 21, 2025 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants