Skip to content

Conversation

@Rooffeell
Copy link

What kind of change does this PR introduce?

  • Bug fix

What is the current behavior?

When test OTP is enabled without an SMS provider configured, the CLI shows the warning:
"WARN: no SMS provider is enabled. Disabling phone login."

This can be confusing for users trying to test OTP flows.

What is the new behavior?

  • The CLI now handles the scenario gracefully:
    • Test OTP can be enabled without requiring an SMS provider.
    • Warning is only shown when relevant to misconfigured production providers.

Additional context

This improves the developer experience when using test OTP flows in Supabase CLI.

… up when test otp is enabled but provider not configured
@Rooffeell Rooffeell requested a review from a team as a code owner January 5, 2026 13:08
@coveralls
Copy link

Pull Request Test Coverage Report for Build 20796811620

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 7 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.03%) to 56.016%

Files with Coverage Reduction New Missed Lines %
internal/storage/rm/rm.go 2 80.61%
internal/gen/keys/keys.go 5 12.9%
Totals Coverage Status
Change from base Build 20786687458: -0.03%
Covered Lines: 6848
Relevant Lines: 12225

💛 - Coveralls

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.

2 participants