支付，OSS 服务重构完成

api/service/sms/aliyun.go

@@ -10,36 +10,57 @@ package sms
 import (
 	"fmt"
 	"geekai/core/types"
+
 	"github.com/aliyun/alibaba-cloud-sdk-go/services/dysmsapi"
 )
 
 type AliYunSmsService struct {
 	config *types.SmsConfigAli
 	client *dysmsapi.Client
+	domain string
+	zoneId string
 }
 
-func NewAliYunSmsService(appConfig *types.AppConfig) (*AliYunSmsService, error) {
-	config := &appConfig.SMS.Ali
-	// 创建阿里云短信客户端
+func NewAliYunSmsService(sysConfig *types.SystemConfig) (*AliYunSmsService, error) {
+	config := &sysConfig.SMS.Ali
+	domain := "dysmsapi.aliyuncs.com"
+	zoneId := "cn-hangzhou"
+
+	s := AliYunSmsService{
+		config: config,
+		domain: domain,
+		zoneId: zoneId,
+	}
+	if sysConfig.SMS.Active == Ali {
+		err := s.UpdateConfig(config)
+		if err != nil {
+			logger.Errorf("阿里云短信初始化失败: %v", err)
+		}
+	}
+	return &s, nil
+}
+
+func (s *AliYunSmsService) UpdateConfig(config *types.SmsConfigAli) error {
 	client, err := dysmsapi.NewClientWithAccessKey(
-		"cn-hangzhou",
+		s.zoneId,
 		config.AccessKey,
 		config.AccessSecret)
 	if err != nil {
-		return nil, fmt.Errorf("failed to create client: %v", err)
+		return fmt.Errorf("failed to create client: %v", err)
 	}
-
-	return &AliYunSmsService{
-		config: config,
-		client: client,
-	}, nil
+	s.client = client
+	s.config = config
+	return nil
 }
 
 func (s *AliYunSmsService) SendVerifyCode(mobile string, code int) error {
+	if s.client == nil {
+		return fmt.Errorf("阿里云短信服务未初始化")
+	}
 	// 创建短信请求并设置参数
 	request := dysmsapi.CreateSendSmsRequest()
 	request.Scheme = "https"
-	request.Domain = s.config.Domain
+	request.Domain = s.domain
 	request.PhoneNumbers = mobile
 	request.SignName = s.config.Sign
 	request.TemplateCode = s.config.CodeTempId

api/service/sms/bao.go

@@ -20,19 +20,20 @@ import (
 
 type BaoSmsService struct {
 	config *types.SmsConfigBao
+	domain string
 }
 
-func NewSmsBaoSmsService(appConfig *types.AppConfig) *BaoSmsService {
-	config := appConfig.SMS.Bao
-	if config.Domain == "" { // use default domain
-		config.Domain = "api.smsbao.com"
-		logger.Infof("Using default domain for SMS-BAO: %s", config.Domain)
-	}
+func NewBaoSmsService(sysConfig *types.SystemConfig) *BaoSmsService {
 	return &BaoSmsService{
-		config: &config,
+		config: &sysConfig.SMS.Bao,
+		domain: "api.smsbao.com",
 	}
 }
 
+func (s *BaoSmsService) UpdateConfig(config *types.SmsConfigBao) {
+	s.config = config
+}
+
 var errMsg = map[string]string{
 	"0":  "短信发送成功",
 	"-1": "参数不全",
@@ -56,7 +57,7 @@ func (s *BaoSmsService) SendVerifyCode(mobile string, code int) error {
 	params.Set("m", mobile)
 	params.Set("c", content)
 
-	apiURL := fmt.Sprintf("https://%s/sms?%s", s.config.Domain, params.Encode())
+	apiURL := fmt.Sprintf("https://%s/sms?%s", s.domain, params.Encode())
 	response, err := http.Get(apiURL)
 	if err != nil {
 		return err

api/service/sms/service_manager.go

@@ -10,37 +10,32 @@ package sms
 import (
 	"geekai/core/types"
 	logger2 "geekai/logger"
-	"strings"
 )
 
-type ServiceManager struct {
-	handler Service
+type SmsManager struct {
+	aliyun *AliYunSmsService
+	bao    *BaoSmsService
+	active string
 }
 
 var logger = logger2.GetLogger()
 
-func NewSendServiceManager(config *types.AppConfig) (*ServiceManager, error) {
-	active := Ali
-	if config.SMS.Active != "" {
-		active = strings.ToUpper(config.SMS.Active)
-	}
-	var handler Service
-	switch active {
-	case Ali:
-		client, err := NewAliYunSmsService(config)
-		if err != nil {
-			return nil, err
-		}
-		handler = client
-		break
-	case Bao:
-		handler = NewSmsBaoSmsService(config)
-		break
-	}
+func NewSmsManager(sysConfig *types.SystemConfig, aliyun *AliYunSmsService, bao *BaoSmsService) (*SmsManager, error) {
 
-	return &ServiceManager{handler: handler}, nil
+	return &SmsManager{
+		active: sysConfig.SMS.Active,
+		aliyun: aliyun,
+		bao:    bao,
+	}, nil
 }
 
-func (m *ServiceManager) GetService() Service {
-	return m.handler
+func (m *SmsManager) GetService() Service {
+	if m.active == Ali {
+		return m.aliyun
+	}
+	return m.bao
+}
+
+func (m *SmsManager) SetActive(active string) {
+	m.active = active
 }