Skip to content
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

feat: mailer logging #1805

Merged
merged 4 commits into from
Oct 15, 2024
Merged

feat: mailer logging #1805

merged 4 commits into from
Oct 15, 2024

Conversation

cstockton
Copy link
Contributor

This is a quick patch to add basic logging information to the mailer package.

I wasn't sure how to only log this information when using the Supabase default mail provider without adding another custom config flag to enable it.

This is a quick patch to add basic logging information to the
mailer package.

I wasn't sure how to only log this information when using the
supabase default mail provider without adding another custom
config flag to enable it.
@cstockton cstockton requested a review from a team as a code owner October 14, 2024 23:35
@coveralls
Copy link

coveralls commented Oct 14, 2024

Pull Request Test Coverage Report for Build 11348418889

Details

  • 7 of 32 (21.88%) changed or added relevant lines in 4 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.04%) to 57.189%

Changes Missing Coverage Covered Lines Changed/Added Lines %
internal/mailer/template.go 6 7 85.71%
internal/mailer/mailer.go 0 9 0.0%
internal/mailer/mailme.go 0 15 0.0%
Totals Coverage Status
Change from base Build 11334620572: -0.04%
Covered Lines: 9550
Relevant Lines: 16699

💛 - Coveralls

internal/api/middleware.go Outdated Show resolved Hide resolved
internal/mailer/mailme.go Outdated Show resolved Hide resolved
internal/mailer/mailme.go Outdated Show resolved Hide resolved
internal/mailer/mailme.go Outdated Show resolved Hide resolved
@cstockton cstockton merged commit 9354b83 into master Oct 15, 2024
2 checks passed
@cstockton cstockton deleted the cs/feat-mailer-logging branch October 15, 2024 16:16
cstockton pushed a commit that referenced this pull request Oct 15, 2024
🤖 I have created a release *beep* *boop*
---


##
[2.163.0](v2.162.2...v2.163.0)
(2024-10-15)


### Features

* add mail header support via `GOTRUE_SMTP_HEADERS` with `$messageType`
([#1804](#1804))
([99d6a13](99d6a13))
* add MFA for WebAuthn
([#1775](#1775))
([8cc2f0e](8cc2f0e))
* configurable email and sms rate limiting
([#1800](#1800))
([5e94047](5e94047))
* mailer logging ([#1805](#1805))
([9354b83](9354b83))
* preserve rate limiters in memory across configuration reloads
([#1792](#1792))
([0a3968b](0a3968b))


### Bug Fixes

* add twilio verify support on mfa
([#1714](#1714))
([aeb5d8f](aeb5d8f))
* email header setting no longer misleading
([#1802](#1802))
([3af03be](3af03be))
* enforce authorized address checks on send email only
([#1806](#1806))
([c0c5b23](c0c5b23))
* fix `getExcludedColumns` slice allocation
([#1788](#1788))
([7f006b6](7f006b6))
* Fix reqPath for bypass check for verify EP
([#1789](#1789))
([646dc66](646dc66))
* inline mailme package for easy development
([#1803](#1803))
([fa6f729](fa6f729))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants