supabase / auth / 10605980831

69%

Build Type

push

github

Committed by web-flow

Commit Message

fix: apply shared limiters before email / sms is sent (#1748)

## What kind of change does this PR introduce?
* Fixes https://github.com/supabase/auth/issues/1236
* Reduces the number of false positives arising from validation errors
when counting rate limits for emails / sms sent
* This change applies the shared rate limiter for email and phone
functions before the actual email / sms is being sent out rather than at
the start of the request
* The `limitEmailOrPhoneSentHandler()` now initialises the rate limiters
and sets it in the request context so we can subsequently retrieve it
right before the email / sms is sent

## What is the current behavior?

Please link any relevant issues here.

## What is the new behavior?

Feel free to include screenshots if it includes visual changes.

## Additional context

Add any other context or screenshots.

Run Details

53 of 106 new or added lines in 12 files covered. (50.0%)

16 existing lines in 1 file now uncovered.

9123 of 15765 relevant lines covered (57.87%)

55.64 hits per line