-
-
Notifications
You must be signed in to change notification settings - Fork 4.8k
Add afterFind
to auth adapter
#8440
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
Comments
Thanks for opening this issue!
|
Could you give more details about the issue? In what scenario would one want to strip authData fields and from what should they be stripped? |
For the MFA Adapter, the secret which is used to generate the OTP is only meant to be seen on the server side. |
Not sure if this can end up causing issues, but be mindful that most of the handling of |
Yeah, I don’t see it being implemented on many with adapters, but for the MFA one we need to store a secret in auth data which shouldn’t be seen on the client side. The client SDKs already need work to support the new auth adapters |
🎉 This change has been released in version 6.1.0-alpha.3 |
🎉 This change has been released in version 6.1.0-beta.2 |
🎉 This change has been released in version 6.1.0 |
New Feature / Enhancement Checklist
Current Limitation
Currently, to strip authData fields, it needs to be done in an afterFind trigger
Feature / Enhancement Description
Give authAdapters the ability to customize find response
Example Use Case
Alternatives / Workarounds
3rd Party References
The text was updated successfully, but these errors were encountered: