Comments (7)
@gagoit I'm not able to replicate this on a brand new 7.1 app:
I'm still looking, it's possible something fundamental has changed somewhere, but wanted to mention the above in case I'm missing something from your example. Thanks.
from devise.
I also faced to this issue, and I saw that came from this change. The hash#deep_merge has been changed behavior:
So when we use Devise.with_options with multiple blocks inside (for adding multiple modules), the options that passed to each module has been merged, that make the Devise::STRATEGIES
has been changed in-correctly => warden has failed to load the incorrect strategies
NOTE:
- In rails 7.0.8, the Devise::STRATEGIES won't include
trackable, timeoutable, lockable...
- In rails 7.1.0, the Devise::STRATEGIES will include
trackable, timeoutable, lockable...
from devise.
@gagoit thanks for the feedback, glad to know the issue is elsewhere :)
@jwoodrow are you also using the same config
gem? It sounds like this could be the culprit.
I tested a brand new app with devise (using this newly created 4-stable branch which contains tweaks for Rails 7.1), and everything appears to be working. If anyone would like to give that a try, let me know if you run into any issues.
from devise.
@carlosantoniodasilva We are indeed using the config gem, guessing the culprit must be there. Thanks to both of you for looking into this and finding the culprit. Now the question for me is, do I truly still need this gem :D
from devise.
Thanks for reporting back. I'll consider this one as closed then for now, but please report back if you run into anything else.
from devise.
@carlosantoniodasilva Thanks for your info.
I've checked the other gems in my project, and saw it's related to the config gem (more details here rails/rails#49457 and rubyconfig/config#342).
Summary:
Config gem uses deep_merge and DeepMerge's Hash monkey patch is not compatible with Rails 7.1 (see rails/rails#49457).
=> that make the behavior of deep_merge
is changed in my project
=> Devise.with_options doesn't work correctly.
Sorry for making you confuse.
from devise.
Thanks for the heads up, for other users:
In order to update rails from 7.0
to 7.1
I had to manually upgrade config gem (from 4.1
Β to 5.1
in my case) which stopped the error Invalid strategy trackable
(was giving error for all strategies, not just trackable
).
from devise.
Related Issues (20)
- Password Reset Workflow Question π HOT 1
- Deprecation - Rails.application.secrets HOT 6
- undefines method utc in reset_password_period_valid? method HOT 3
- How to Expire Session IDs in Cookies for Active Admin?
- DEPRECATION WARNING: `Rails.application.secrets` is deprecated in favor of `Rails.application.credentials` and will be removed in Rails 7.2. HOT 2
- Possibly incorrect skip_trackable check for timeoutable
- strong params not working as per "lazy way" instructions
- Sorbet support for Devise Controllers
- ActionController::InvalidAuthenticityToken in Devise::SessionsController#create HOT 14
- "Encrypted password" is actually "Hash" of the password. Does it need to be renamed?
- Error during failsafe response: uncaught throw :warden HOT 1
- `confirmation_sent_at` is set regardless `skip_confirmation_notification!` method call HOT 1
- Resend confirmation instructions triggering error HOT 1
- Replace [data-turbo-cache=false] with [data-turbo-temporary]
- `parent_mailer` config option boot error
- Converting views from Erb to Haml causes them to be ignored by Devise HOT 1
- How to prevent to send a confirmation email when email field is updated ? #903 HOT 1
- Active Job integration clarification HOT 1
- user.update_with_password ignores password_confirmation
- Deprecation warning about Rails.application.secrets on rails 7.1 HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
π Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. πππ
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google β€οΈ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from devise.