fix: also adds max frquency error

awalias · awalias · commit 14137e820ce3 · 2020-11-13T13:18:28.000+08:00
diff --git a/api/mail.go b/api/mail.go
@@ -17,7 +17,7 @@ var (
 
 func sendConfirmation(tx *storage.Connection, u *models.User, mailer mailer.Mailer, maxFrequency time.Duration, referrerURL string) error {
 	if u.ConfirmationSentAt != nil && !u.ConfirmationSentAt.Add(maxFrequency).Before(time.Now()) {
-		return nil
+		return MaxFrequencyLimitError
 	}
 
 	oldToken := u.ConfirmationToken
@@ -45,7 +45,7 @@ func sendInvite(tx *storage.Connection, u *models.User, mailer mailer.Mailer, re
 
 func (a *API) sendPasswordRecovery(tx *storage.Connection, u *models.User, mailer mailer.Mailer, maxFrequency time.Duration, referrerURL string) error {
 	if u.RecoverySentAt != nil && !u.RecoverySentAt.Add(maxFrequency).Before(time.Now()) {
-		return nil
+		return MaxFrequencyLimitError
 	}
 
 	oldToken := u.RecoveryToken

Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ var (`
`17`	`17`
`18`	`18`	`func sendConfirmation(tx storage.Connection, u models.User, mailer mailer.Mailer, maxFrequency time.Duration, referrerURL string) error {`
`19`	`19`	`if u.ConfirmationSentAt != nil && !u.ConfirmationSentAt.Add(maxFrequency).Before(time.Now()) {`
`20`		`- return nil`
	`20`	`+ return MaxFrequencyLimitError`
`21`	`21`	`}`
`22`	`22`
`23`	`23`	`oldToken := u.ConfirmationToken`
`@@ -45,7 +45,7 @@ func sendInvite(tx storage.Connection, u models.User, mailer mailer.Mailer, re`
`45`	`45`
`46`	`46`	`func (a API) sendPasswordRecovery(tx storage.Connection, u *models.User, mailer mailer.Mailer, maxFrequency time.Duration, referrerURL string) error {`
`47`	`47`	`if u.RecoverySentAt != nil && !u.RecoverySentAt.Add(maxFrequency).Before(time.Now()) {`
`48`		`- return nil`
	`48`	`+ return MaxFrequencyLimitError`
`49`	`49`	`}`
`50`	`50`
`51`	`51`	`oldToken := u.RecoveryToken`