fix(fastify): use unprefixed email in duplicate-email warning body#204
Open
semantabhandari wants to merge 1 commit into
Open
fix(fastify): use unprefixed email in duplicate-email warning body#204semantabhandari wants to merge 1 commit into
semantabhandari wants to merge 1 commit into
Conversation
The duplicate-email registration warning rendered the internal
tenant-namespaced address (<accountId>_<email>) in the email body
instead of the clean email the user typed. In the emailPasswordSignUp
override the email is namespaced per tenant before storage, and the
warning's templateData.emailId was set from the prefixed input.email
rather than the original clean value. The `to` field was already
correct.
Pass originalEmail to templateData.emailId so {{emailId}} renders the
raw address. Adds a test asserting the warning's templateData.emailId
equals the raw email with no <accountId>_ prefix, and bumps the version
to 0.29.2.
uddhab
approved these changes
Jul 7, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The duplicate-email registration warning rendered the internal tenant-namespaced address (_) in the email body instead of the clean email the user typed. In the emailPasswordSignUp override the email is namespaced per tenant before storage, and the warning's templateData.emailId was set from the prefixed input.email rather than the original clean value. The
tofield was already correct.Pass originalEmail to templateData.emailId so {{emailId}} renders the raw address. Adds a test asserting the warning's templateData.emailId equals the raw email with no _ prefix, and bumps the version to 0.29.2.