Kharec/goyco
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Compare commits

base: Kharec:6643466d769c1de4b9309790b87a30369eba52e4
Branches Tags
Kharec:main
...
compare: Kharec:a1466e860dc92e8f01e0b44be315f5823cbaf92d
Branches Tags
Kharec:main

3 Commits
6643466d76 ... a1466e860d

Author SHA1 Message Date
Kharec
a1466e860d refactor: remove redundant validation, trust middleware and service layer 2026-01-10 22:41:54 +01:00
Kharec
a1e63b868f refactor: remove redundant validation and unused import 2026-01-10 22:41:43 +01:00
Kharec
b6f5293c0f refactor: remove redundant validation 2026-01-10 22:41:31 +01:00
3 changed files with 0 additions and 63 deletions
Expand all files Collapse all files

37
internal/handlers/auth_handler.go
View File

@@ -124,20 +124,6 @@ func (h *AuthHandler) Register(w http.ResponseWriter, r *http.Request) {
password := strings.TrimSpace(req.Password) password := strings.TrimSpace(req.Password)
username = security.SanitizeUsername(username) username = security.SanitizeUsername(username)
if err := validation.ValidateUsername(username); err != nil {
SendErrorResponse(w, err.Error(), http.StatusBadRequest)
return
}
if err := validation.ValidateEmail(email); err != nil {
SendErrorResponse(w, err.Error(), http.StatusBadRequest)
return
}
if err := validation.ValidatePassword(password); err != nil {
SendErrorResponse(w, err.Error(), http.StatusBadRequest)
return
}
result, err := h.authService.Register(username, email, password) result, err := h.authService.Register(username, email, password)
if err != nil { if err != nil {
@@ -313,11 +299,6 @@ func (h *AuthHandler) ResetPassword(w http.ResponseWriter, r *http.Request) {
return return
} }
if err := validation.ValidatePassword(newPassword); err != nil {
SendErrorResponse(w, err.Error(), http.StatusBadRequest)
return
}
if err := h.authService.ResetPassword(token, newPassword); err != nil { if err := h.authService.ResetPassword(token, newPassword); err != nil {
switch { switch {
case strings.Contains(err.Error(), "expired"): case strings.Contains(err.Error(), "expired"):
@@ -360,10 +341,6 @@ func (h *AuthHandler) UpdateEmail(w http.ResponseWriter, r *http.Request) {
} }
email := strings.TrimSpace(req.Email) email := strings.TrimSpace(req.Email)
if err := validation.ValidateEmail(email); err != nil {
SendErrorResponse(w, err.Error(), http.StatusBadRequest)
return
}
user, err := h.authService.UpdateEmail(userID, email) user, err := h.authService.UpdateEmail(userID, email)
if err != nil { if err != nil {
@@ -410,10 +387,6 @@ func (h *AuthHandler) UpdateUsername(w http.ResponseWriter, r *http.Request) {
} }
username := strings.TrimSpace(req.Username) username := strings.TrimSpace(req.Username)
if err := validation.ValidateUsername(username); err != nil {
SendErrorResponse(w, err.Error(), http.StatusBadRequest)
return
}
user, err := h.authService.UpdateUsername(userID, username) user, err := h.authService.UpdateUsername(userID, username)
if err != nil { if err != nil {
@@ -457,16 +430,6 @@ func (h *AuthHandler) UpdatePassword(w http.ResponseWriter, r *http.Request) {
currentPassword := strings.TrimSpace(req.CurrentPassword) currentPassword := strings.TrimSpace(req.CurrentPassword)
newPassword := strings.TrimSpace(req.NewPassword) newPassword := strings.TrimSpace(req.NewPassword)
if currentPassword == "" {
SendErrorResponse(w, "Current password is required", http.StatusBadRequest)
return
}
if err := validation.ValidatePassword(newPassword); err != nil {
SendErrorResponse(w, err.Error(), http.StatusBadRequest)
return
}
user, err := h.authService.UpdatePassword(userID, currentPassword, newPassword) user, err := h.authService.UpdatePassword(userID, currentPassword, newPassword)
if err != nil { if err != nil {
if strings.Contains(err.Error(), "current password is incorrect") { if strings.Contains(err.Error(), "current password is incorrect") {

11
internal/handlers/post_handler.go
View File

@@ -12,7 +12,6 @@ import (
"goyco/internal/repositories" "goyco/internal/repositories"
"goyco/internal/security" "goyco/internal/security"
"goyco/internal/services" "goyco/internal/services"
"goyco/internal/validation"
"github.com/go-chi/chi/v5" "github.com/go-chi/chi/v5"
"github.com/jackc/pgconn" "github.com/jackc/pgconn"
@@ -273,16 +272,6 @@ func (h *PostHandler) UpdatePost(w http.ResponseWriter, r *http.Request) {
title := security.SanitizeInput(req.Title) title := security.SanitizeInput(req.Title)
content := security.SanitizePostContent(req.Content) content := security.SanitizePostContent(req.Content)
if err := validation.ValidateTitle(title); err != nil {
SendErrorResponse(w, err.Error(), http.StatusBadRequest)
return
}
if err := validation.ValidateContent(content); err != nil {
SendErrorResponse(w, err.Error(), http.StatusBadRequest)
return
}
post.Title = title post.Title = title
post.Content = content post.Content = content

15
internal/handlers/user_handler.go
View File

@@ -99,21 +99,6 @@ func (h *UserHandler) CreateUser(w http.ResponseWriter, r *http.Request) {
return return
} }
if err := validation.ValidateUsername(req.Username); err != nil {
SendErrorResponse(w, err.Error(), http.StatusBadRequest)
return
}
if err := validation.ValidateEmail(req.Email); err != nil {
SendErrorResponse(w, err.Error(), http.StatusBadRequest)
return
}
if err := validation.ValidatePassword(req.Password); err != nil {
SendErrorResponse(w, err.Error(), http.StatusBadRequest)
return
}
result, err := h.authService.Register(req.Username, req.Email, req.Password) result, err := h.authService.Register(req.Username, req.Email, req.Password)
if err != nil { if err != nil {
var validationErr *validation.ValidationError var validationErr *validation.ValidationError