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
fix: u&p service hashes passwords #20147
fix: u&p service hashes passwords #20147
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Indeed can we please switch to the docsrv |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works well for me, thanks!
We're still discussing how to handle the API side of things, some parts may need to wait until Strapi 6. For now we'll go with this because it needs to be fixed ASAP, but internally we can refactor at any time to use doc service once we expose the entity id api. I do also hope we have time to use doc id for the u&p content api as well. |
What does it do?
The u&p service lost its password hashing during the deprecation migration of entity service. This adds it back.
We may actually want to switch to use document service universally here, in case there's any other logic we're missing and to avoid duplicating code.
Why is it needed?
u&p passwords aren't being hashed on registration
How to test it?
Tests have been added, but:
Related issue(s)/PR(s)
DX-1372