supabase / auth / 26461355138

71%

push

github

fix(mailer): include SiteURL in notification template data (#2532)

## Summary

The seven notification mail builders (`PasswordChanged`, `EmailChanged`,
`PhoneChanged`, `IdentityLinked`, `IdentityUnlinked`,
`MFAFactorEnrolled`, `MFAFactorUnenrolled`) omitted `SiteURL` from their
template data maps, so templates referencing `{{ .SiteURL }}` render an
empty string at runtime. The standard mail builders (`Invite`,
`Confirmation`, `Recovery`, `MagicLink`, `EmailChange`,
`Reauthentication`) already include it, and `checkDefaults` in
`template.go` already whitelists `SiteURL` for all template types — so
this looks like an oversight from when notification templates were
added, not an intentional restriction.

Closes #2468

## Test plan

- [x] `go build ./...` passes
- [x] `go test ./internal/mailer/templatemailer/...` passes
- [x] `gofmt` clean

Co-authored-by: Chris Stockton <180184+cstockton@users.noreply.github.com>

0 of 9 new or added lines in 1 file covered. (0.0%)

17160 of 24225 relevant lines covered (70.84%)

83.38 hits per line