fix: some bugs about SMS API (#310)

Signed-off-by: “seriouszyx” <674965440@qq.com>
This commit is contained in:
Yixiang Zhao
2021-10-31 08:49:39 +08:00
committed by GitHub
parent b7f2f9056f
commit 609e9785e4
3 changed files with 10 additions and 6 deletions

View File

@ -96,6 +96,7 @@ func (c *ApiController) SendSms() {
var smsForm struct {
Content string `json:"content"`
Receivers []string `json:"receivers"`
OrgId string `json:"organizationId"` // e.g. "admin/built-in"
}
err := json.Unmarshal(c.Ctx.Input.RequestBody, &smsForm)
if err != nil {
@ -103,10 +104,13 @@ func (c *ApiController) SendSms() {
return
}
org := object.GetOrganization(smsForm.OrgId)
var invalidReceivers []string
for _, receiver := range smsForm.Receivers {
for idx, receiver := range smsForm.Receivers {
if !util.IsPhoneCnValid(receiver) {
invalidReceivers = append(invalidReceivers, receiver)
} else {
smsForm.Receivers[idx] = fmt.Sprintf("+%s%s", org.PhonePrefix, receiver)
}
}

View File

@ -63,7 +63,7 @@ func (c *ApiController) SendVerificationCode() {
user := c.getCurrentUser()
organization := object.GetOrganization(orgId)
application := object.GetApplicationByOrganizationName(organization.Name)
sendResp := errors.New("Invalid dest type.")
switch destType {
case "email":
@ -90,12 +90,12 @@ func (c *ApiController) SendVerificationCode() {
sendResp = object.SendVerificationCodeToPhone(organization, user, provider, remoteAddr, dest)
}
status := "ok"
if sendResp != nil {
status = "error"
c.Data["json"] = Response{Status: "error", Msg: sendResp.Error()}
} else {
c.Data["json"] = Response{Status: "ok"}
}
c.Data["json"] = Response{Status: status, Msg: sendResp.Error()}
c.ServeJSON()
}

View File

@ -68,7 +68,7 @@ func SendVerificationCodeToPhone(organization *Organization, user *User, provide
return err
}
return SendSms(provider, dest, code)
return SendSms(provider, code, dest)
}
func AddToVerificationRecord(user *User, provider *Provider, remoteAddr, recordType, dest, code string) error {