Compare commits
No commits in common. "ea5a97c6137da57ef34aa923fddc11fe2a4638d2" and "12a9ad28fdbea9b55da3b9a7f047f6407c2b50e0" have entirely different histories.
ea5a97c613
...
12a9ad28fd
|
@ -134,10 +134,6 @@ func (s AccountServiceDefault) SendEmailVerification(userId uint) error {
|
||||||
return err
|
return err
|
||||||
}
|
}
|
||||||
|
|
||||||
if user.Verified {
|
|
||||||
return NewAccountError(ErrKeyAccountAlreadyVerified, nil)
|
|
||||||
}
|
|
||||||
|
|
||||||
token := GenerateSecurityToken()
|
token := GenerateSecurityToken()
|
||||||
|
|
||||||
var verification models.EmailVerification
|
var verification models.EmailVerification
|
||||||
|
@ -202,7 +198,7 @@ func (s AccountServiceDefault) VerifyEmail(email string, token string) error {
|
||||||
|
|
||||||
if result.Error != nil {
|
if result.Error != nil {
|
||||||
if errors.Is(result.Error, gorm.ErrRecordNotFound) {
|
if errors.Is(result.Error, gorm.ErrRecordNotFound) {
|
||||||
return NewAccountError(ErrKeyEmailVerificationFailed, result.Error)
|
return NewAccountError(ErrKeyUserNotFound, result.Error)
|
||||||
}
|
}
|
||||||
|
|
||||||
return NewAccountError(ErrKeyDatabaseOperationFailed, result.Error)
|
return NewAccountError(ErrKeyDatabaseOperationFailed, result.Error)
|
||||||
|
|
|
@ -26,8 +26,6 @@ const (
|
||||||
|
|
||||||
// Account update errors
|
// Account update errors
|
||||||
ErrKeyAccountUpdateFailed = "ErrAccountUpdateFailed"
|
ErrKeyAccountUpdateFailed = "ErrAccountUpdateFailed"
|
||||||
ErrKeyAccountAlreadyVerified = "ErrAccountAlreadyVerified"
|
|
||||||
ErrKeyEmailVerificationFailed = "ErrEmailVerificationFailed"
|
|
||||||
|
|
||||||
// JWT generation errors
|
// JWT generation errors
|
||||||
ErrKeyJWTGenerationFailed = "ErrJWTGenerationFailed"
|
ErrKeyJWTGenerationFailed = "ErrJWTGenerationFailed"
|
||||||
|
@ -75,8 +73,6 @@ var defaultErrorMessages = map[string]string{
|
||||||
|
|
||||||
// Account update errors
|
// Account update errors
|
||||||
ErrKeyAccountUpdateFailed: "Failed to update account information.",
|
ErrKeyAccountUpdateFailed: "Failed to update account information.",
|
||||||
ErrKeyAccountAlreadyVerified: "Account is already verified.",
|
|
||||||
ErrKeyEmailVerificationFailed: "Failed to verify email address.",
|
|
||||||
|
|
||||||
// JWT generation errors
|
// JWT generation errors
|
||||||
ErrKeyJWTGenerationFailed: "Failed to generate a new JWT token.",
|
ErrKeyJWTGenerationFailed: "Failed to generate a new JWT token.",
|
||||||
|
@ -123,8 +119,6 @@ var (
|
||||||
|
|
||||||
// Account update errors
|
// Account update errors
|
||||||
ErrKeyAccountUpdateFailed: http.StatusInternalServerError,
|
ErrKeyAccountUpdateFailed: http.StatusInternalServerError,
|
||||||
ErrKeyAccountAlreadyVerified: http.StatusConflict,
|
|
||||||
ErrKeyEmailVerificationFailed: http.StatusInternalServerError,
|
|
||||||
|
|
||||||
// JWT generation errors
|
// JWT generation errors
|
||||||
ErrKeyJWTGenerationFailed: http.StatusInternalServerError,
|
ErrKeyJWTGenerationFailed: http.StatusInternalServerError,
|
||||||
|
|
Loading…
Reference in New Issue