-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Add support for custom URLs for email verification #22565
Conversation
🦋 Changeset detectedLatest commit: 47d6cf3 The changes in this PR will be included in the next version bump. This PR includes changesets to release 9 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Feel free to merge - the rest isn't documented anyway. I'm working on it and will incorporate this ;) |
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.
🎉
Co-authored-by: Pascal Jufer <[email protected]>
Hi Team, I'm encountering an issue with the recent update that adds support for custom URLs in email verification. Despite the fix being merged and the verification_url now included, I still receive an error indicating an invalid payload. Here are the details:
Context Steps Taken
Potential Issues
Request for Assistance Thank you for your attention to this matter. Looking forward to your guidance on resolving this issue. |
Hey @shubham-am-i it sounds like youre not on the latest version For example:
And then an API request like the following: curljson \
-d '{ "email": "[email protected]", "password": "test", "verification_url": "http://asd.de" }' \
<YOUR-DIRECTUS-URL-HERE>/users/register works and correctly uses the verification URL inside of the email. Please verify that youre on the correct Directus version and are POSTing to the correct URL If you have further troubles please provide your settings and api request in a new issue instead of this PR - Thank you! :) |
Scope
What's changed:
USER_REGISTER_URL_ALLOW_LIST
similar toUSER_INVITE_URL_ALLOW_LIST
Potential Risks / Drawbacks
Review Notes / Questions
USER_VERIFY_URL_ALLOW_LIST
to make it more clear that this is pertaining to verification and not registration for example but thats a minor thingFixes #22561