all repos

onasty @ c9bc2f83b8df50e6713d59b5874e1d2c1a09d053

a one-time notes service
4 files changed, 17 insertions(+), 13 deletions(-)
refactor: common response for invalid bodies (#203)

Author: Oleksandr Smirnov olexsmir@gmail.com
Committed by: GitHub noreply@github.com
Committed at: 2025-09-01 13:05:35 +0300
Parent: 312d08f
M internal/transport/http/apiv1/auth.go

@@ -16,7 +16,7 @@

func (a *APIV1) signUpHandler(c *gin.Context) { var req signUpRequest if err := c.ShouldBindJSON(&req); err != nil { - newError(c, http.StatusBadRequest, "invalid request") + invalidRequest(c) return }

@@ -46,7 +46,7 @@

func (a *APIV1) signInHandler(c *gin.Context) { var req signInRequest if err := c.ShouldBindJSON(&req); err != nil { - newError(c, http.StatusBadRequest, "invalid request") + invalidRequest(c) return }

@@ -72,7 +72,7 @@

func (a *APIV1) refreshTokensHandler(c *gin.Context) { var req refreshTokenRequest if err := c.ShouldBindJSON(&req); err != nil { - newError(c, http.StatusBadRequest, "invalid request") + invalidRequest(c) return }

@@ -95,7 +95,7 @@

func (a *APIV1) logOutHandler(c *gin.Context) { var req logoutRequest if err := c.ShouldBindJSON(&req); err != nil { - newError(c, http.StatusBadRequest, "invalid request") + invalidRequest(c) return }
M internal/transport/http/apiv1/note.go

@@ -24,7 +24,7 @@

func (a *APIV1) createNoteHandler(c *gin.Context) { var req createNoteRequest if err := c.ShouldBindJSON(&req); err != nil { - newError(c, http.StatusBadRequest, "invalid request") + invalidRequest(c) return }

@@ -87,7 +87,7 @@

func (a *APIV1) getNoteBySlugAndPasswordHandler(c *gin.Context) { var req getNoteBuySlugAndPasswordRequest if err := c.ShouldBindJSON(&req); err != nil { - newError(c, http.StatusBadRequest, "invalid request") + invalidRequest(c) return }

@@ -183,7 +183,7 @@

func (a *APIV1) updateNoteHandler(c *gin.Context) { var req updateNoteRequest if err := c.ShouldBindJSON(&req); err != nil { - newError(c, http.StatusBadRequest, "invalid request") + invalidRequest(c) return }

@@ -223,7 +223,7 @@

func (a *APIV1) setNotePasswordHandler(c *gin.Context) { var req setNotePasswordRequest if err := c.ShouldBindJSON(&req); err != nil { - newError(c, http.StatusBadRequest, "invalid request") + invalidRequest(c) return }
M internal/transport/http/apiv1/response.go

@@ -59,6 +59,10 @@

newInternalError(c, err) } +func invalidRequest(c *gin.Context) { + newError(c, http.StatusBadRequest, "invalid request") +} + func newError(c *gin.Context, status int, msg string) { slog.ErrorContext(c.Request.Context(), msg, "status", status) c.AbortWithStatusJSON(status, response{msg})
M internal/transport/http/apiv1/user.go

@@ -38,7 +38,7 @@

func (a *APIV1) changePasswordHandler(c *gin.Context) { var req changePasswordRequest if err := c.ShouldBindJSON(&req); err != nil { - newError(c, http.StatusBadRequest, "invalid request") + invalidRequest(c) return }

@@ -63,7 +63,7 @@

func (a *APIV1) requestResetPasswordHandler(c *gin.Context) { var req requestResetPasswordRequest if err := c.ShouldBindJSON(&req); err != nil { - newError(c, http.StatusBadRequest, "invalid request") + invalidRequest(c) return }

@@ -87,7 +87,7 @@

func (a *APIV1) resetPasswordHandler(c *gin.Context) { var req resetPasswordRequest if err := c.ShouldBindJSON(&req); err != nil { - newError(c, http.StatusBadRequest, "invalid request") + invalidRequest(c) return }

@@ -112,7 +112,7 @@

func (a *APIV1) requestEmailChangeHandler(c *gin.Context) { var req changeEmailRequest if err := c.ShouldBindJSON(&req); err != nil { - newError(c, http.StatusBadRequest, "invalid request") + invalidRequest(c) return }

@@ -160,7 +160,7 @@

func (a *APIV1) resendVerificationEmailHandler(c *gin.Context) { var req resendVerificationEmailRequest if err := c.ShouldBindJSON(&req); err != nil { - newError(c, http.StatusBadRequest, "invalid request") + invalidRequest(c) return }