gromov/casdoor
1
0
Fork 0
forked from casdoor/casdoor
Code Pull Requests Actions Packages Releases Activity

feat: fix Twilio SMS sending verification code bug (#5205)

Browse Source
This commit is contained in:
Yang Luo
2026-03-06 22:32:09 +08:00
parent 41ec8ba44f
commit c10b2c162f
1 changed files with 9 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
object/sms.go
View File

@@ -32,6 +32,10 @@ func getSmsClient(provider *Provider) (sender.SmsClient, error) {
client, err = newHttpSmsClient(provider.Endpoint, provider.Method, provider.Title, provider.TemplateCode, provider.HttpHeaders, provider.UserMapping, provider.IssuerUrl, provider.EnableProxy)
} else if provider.Type == "Alibaba Cloud PNVS SMS" {
client, err = newPnvsSmsClient(provider.ClientId, provider.ClientSecret, provider.SignName, provider.TemplateCode, provider.RegionId)
} else if provider.Type == sender.Twilio {
// For Twilio, the message body is pre-formatted in SendSms using the template.
// Pass "%s" as the template so go-sms-sender's fmt.Sprintf passes the content through unchanged.
client, err = sender.NewSmsClient(provider.Type, provider.ClientId, provider.ClientSecret, provider.SignName, "%s", provider.AppId)
} else {
client, err = sender.NewSmsClient(provider.Type, provider.ClientId, provider.ClientSecret, provider.SignName, provider.TemplateCode, provider.AppId)
}
@@ -52,6 +56,11 @@ func SendSms(provider *Provider, content string, phoneNumbers ...string) error {
if provider.AppId != "" {
phoneNumbers = append([]string{provider.AppId}, phoneNumbers...)
}
// Pre-format the message body using the provider's template.
// If the template contains "%s", substitute the code; otherwise use the content (code) directly.
if strings.Contains(provider.TemplateCode, "%s") {
content = strings.Replace(provider.TemplateCode, "%s", content, 1)
}
} else if provider.Type == sender.Aliyun || provider.Type == "Alibaba Cloud PNVS SMS" {
for i, number := range phoneNumbers {
phoneNumbers[i] = strings.TrimPrefix(number, "+86")