Merge pull request #112 from harshithpabbati/signup-msg

fix: return user already exists message on signing up more than once …

Author: awalias

api/signup.go

@@ -88,6 +88,11 @@ func (a *API) Signup(w http.ResponseWriter, r *http.Request) error {
 			mailer := a.Mailer(ctx)
 			referrer := a.getReferrer(r)
 			if terr = sendConfirmation(tx, user, mailer, config.SMTP.MaxFrequency, referrer); terr != nil {
+				if errors.Is(terr, MaxFrequencyLimitError) {
+					now := time.Now()
+					left := user.ConfirmationSentAt.Add(config.SMTP.MaxFrequency).Add(-now)
+					return tooManyRequestsError(fmt.Sprintf("For security purposes, you can only request this after %d seconds.", left))
+				}
 				return internalServerError("Error sending confirmation mail").WithInternalError(terr)
 			}
 		}