cli icon indicating copy to clipboard operation
cli copied to clipboard

Poorly Named Email Enabled Property

Open Crisfole opened this issue 1 year ago • 0 comments

Link

The Docs: https://supabase.com/docs/guides/cli/config#auth.email.enable_signup

The Code: https://github.com/supabase/cli/blob/develop/internal/start/start.go#L403

Describe the problem

Is the documentation missing? Or is it confusing? Why is it confusing?

The documentation is very confusing. It says, "Allow/disallow new user signups via email to your project.", but it is used to determine the value of utils.Config.Auth.Email.EnableSignup, which is actually used to set the GoTrue env variable GOTRUE_EXTERNAL_EMAIL_ENABLED. This is very different from signup enabled.

Describe the improvement

A clear and concise description of the improvement.

Honestly, I think the real solution is a breaking change, unfortunately: updating the variable name (and toml setup) to accurately reflect what the variable does. In the meantime thoroughly documenting the weird difference between the name and behavior would be a great start.

Additional context

Add any other context or screenshots that help clarify your question.

None, really.

Crisfole avatar Mar 17 '24 17:03 Crisfole