WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@jnschaeffer
Copy link
Contributor

What kind of change does this PR introduce?

This PR adds two new metrics, gotrue_email_send_operations_counter_total and gotrue_email_send_errors_counter_total, for tracking email send operations and errors respectively. The purpose of these metrics is to track email delivery independently of the various API handlers that may send email within Auth.

What is the current behavior?

Users who want to track email sending operations in Auth must use indirect metrics like HTTP status codes to determine whether an email was successfully sent.

What is the new behavior?

The gotrue_email_send_operations_counter_total counter will increment each time Auth attempts to send an email after performing validation and checking rate limits. The gotrue_email_send_errors_counter_total counter will increment each time an email send attempt fails.

This commit adds two new metrics,
gotrue_email_send_operations_counter_total and
gotrue_email_send_errors_counter_total, for tracking email send
operations and errors respectively. The purpose of these metrics is to
track email delivery independently of the various API handlers that
may send email within Auth.
@jnschaeffer jnschaeffer requested a review from a team as a code owner January 2, 2026 20:26
@coveralls
Copy link

Pull Request Test Coverage Report for Build 20666067842

Details

  • 2 of 7 (28.57%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.01%) to 68.77%

Changes Missing Coverage Covered Lines Changed/Added Lines %
internal/api/mail.go 2 7 28.57%
Totals Coverage Status
Change from base Build 20599680500: -0.01%
Covered Lines: 14745
Relevant Lines: 21441

💛 - Coveralls

@jnschaeffer jnschaeffer enabled auto-merge (squash) January 2, 2026 21:14
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.

3 participants