diff --git a/controllers/verification.go b/controllers/verification.go index 84785253..91b67c82 100644 --- a/controllers/verification.go +++ b/controllers/verification.go @@ -142,6 +142,10 @@ func (c *ApiController) SendVerificationCode() { c.ResponseError(err.Error()) return } + if provider == nil { + c.ResponseError(fmt.Sprintf("please add an Email provider to the \"Providers\" list for the application: %s", application.Name)) + return + } sendResp = object.SendVerificationCodeToEmail(organization, user, provider, remoteAddr, vform.Dest) case object.VerifyTypePhone: @@ -184,6 +188,10 @@ func (c *ApiController) SendVerificationCode() { c.ResponseError(err.Error()) return } + if provider == nil { + c.ResponseError(fmt.Sprintf("please add a SMS provider to the \"Providers\" list for the application: %s", application.Name)) + return + } if phone, ok := util.GetE164Number(vform.Dest, vform.CountryCode); !ok { c.ResponseError(fmt.Sprintf(c.T("verification:Phone number is invalid in your region %s"), vform.CountryCode)) diff --git a/object/verification.go b/object/verification.go index 7ff8f741..c50f1016 100644 --- a/object/verification.go +++ b/object/verification.go @@ -80,10 +80,6 @@ func IsAllowSend(user *User, remoteAddr, recordType string) error { } func SendVerificationCodeToEmail(organization *Organization, user *User, provider *Provider, remoteAddr string, dest string) error { - if provider == nil { - return fmt.Errorf("please set an Email provider first") - } - sender := organization.DisplayName title := provider.Title code := getRandomCode(6) @@ -106,10 +102,6 @@ func SendVerificationCodeToEmail(organization *Organization, user *User, provide } func SendVerificationCodeToPhone(organization *Organization, user *User, provider *Provider, remoteAddr string, dest string) error { - if provider == nil { - return errors.New("please set a SMS provider first") - } - if err := IsAllowSend(user, remoteAddr, provider.Category); err != nil { return err }