Merge pull request #278 from Jinvic/feat/mail

fix(mail): 验证邮箱是否为空
kingwrcy · Feb 14, 2025 · 4bda6c8 · 4bda6c8
2 parents 4d2c74e + 5df725e
commit 4bda6c8
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/backend/handler/comment.go b/backend/handler/comment.go
@@ -209,8 +209,6 @@ func (c CommentHandler) AddComment(ctx echo.Context) error {
 			}
 			if err = c.commentEmailNotification(comment, frontendHost); err != nil {
 				c.base.log.Error().Msgf("邮件通知失败,原因:%s", err)
-			} else {
-				c.base.log.Info().Msgf("成功发送邮件")
 			}
 		}()
 		return SuccessResp(ctx, h{})
@@ -235,13 +233,16 @@ func (c CommentHandler) commentEmailNotification(comment db.Comment, host string
 		return nil
 	}
 
-	// 验证邮箱是否可用
+	// 验证邮箱是否为空
 	var targetEmail string
 	if comment.ReplyTo != "" { // 回复评论
 		targetEmail = comment.ReplyEmail
 	} else { // 直接评论
 		targetEmail = user.Email
 	}
+	if targetEmail == "" {
+		return nil
+	}
 
 	// 获取smtp客户端
 	client, err := mail.GetSMTPClient(sysConfigVO.SmtpHost, sysConfigVO.SmtpPort, sysConfigVO.SmtpUsername, sysConfigVO.SmtpPassword)
@@ -300,5 +301,6 @@ func (c CommentHandler) commentEmailNotification(comment db.Comment, host string
 		return err
 	}
 
+	c.base.log.Info().Msgf("成功发送邮件")
 	return nil
 }