From 3dd49332d563dbe90a7b52a1ce394f85d1797c00 Mon Sep 17 00:00:00 2001 From: Nilo Roberto C Paim Date: Sun, 21 Aug 2022 07:54:29 -0300 Subject: [PATCH] changed status code; added bat to compile for linux --- compile_linux.bat | 3 ++ controllers/authController.go | 58 +++++++------------------------- controllers/eventController.go | 33 ++++-------------- controllers/versionController.go | 2 +- utils/utils.go | 1 - 5 files changed, 23 insertions(+), 74 deletions(-) create mode 100644 compile_linux.bat diff --git a/compile_linux.bat b/compile_linux.bat new file mode 100644 index 0000000..7ba9eda --- /dev/null +++ b/compile_linux.bat @@ -0,0 +1,3 @@ +set GOARCH=amd64 +set GOOS=linux +go build diff --git a/controllers/authController.go b/controllers/authController.go index 1fb8bc8..66e8526 100644 --- a/controllers/authController.go +++ b/controllers/authController.go @@ -19,7 +19,7 @@ func Login(c *fiber.Ctx) error { var data map[string]string if err := c.BodyParser(&data); err != nil { - return err + return fiber.ErrBadRequest } var user models.User @@ -27,17 +27,11 @@ func Login(c *fiber.Ctx) error { database.DB.Where("email = ?", data["email"]).First(&user) if user.Id == 0 { - return c.JSON(fiber.Map{ - "status": "error", - "message": "Usuário não cadastrado", - }) + return fiber.ErrNotFound } if err := bcrypt.CompareHashAndPassword(user.Password, []byte(data["password"])); err != nil { - return c.JSON(fiber.Map{ - "status": "error", - "message": "Senha incorreta", - }) + return fiber.ErrUnauthorized } type customClaims struct { @@ -57,14 +51,10 @@ func Login(c *fiber.Ctx) error { token, err := claims.SignedString([]byte(os.Getenv("API_SECRET"))) if err != nil { - return c.JSON(fiber.Map{ - "status": "error", - "message": "(A1) - Erro na geração do token", - }) + return fiber.ErrInternalServerError } return c.JSON(fiber.Map{ - "status": "success", "token": token}) } @@ -74,25 +64,19 @@ func AddUser(c *fiber.Ctx) error { var data map[string]string if err := c.BodyParser(&data); err != nil { - return err + return fiber.ErrBadRequest } // If I don't receive an auth field in the request, I need to verify if the sender is logged if data["auth"] == "" { _, err := utils.ProcessToken(c) if err != nil { - return c.JSON(fiber.Map{ - "status": "error", - "message": "Autorização inválida", - }) + return fiber.ErrUnauthorized } } if data["name"] == "" || data["email"] == "" || data["password"] == "" || data["channel"] == "" { - return c.JSON(fiber.Map{ - "status": "error", - "message": "Nome, E-Mail, Senha e Nome do Canal devem ser informados", - }) + return fiber.ErrBadRequest } passwd, _ := utils.HashPassword(data["password"]) @@ -112,14 +96,10 @@ func AddUser(c *fiber.Ctx) error { database.DB.Create(&user) if user.Id == 0 { - return c.JSON(fiber.Map{ - "status": "error", - "message": "Usuário já cadastrado", - }) + return fiber.ErrNotAcceptable } return c.JSON(fiber.Map{ - "status": "success", "user": user, }) } @@ -131,23 +111,16 @@ func GetOwnUser(c *fiber.Ctx) error { userCode, err := utils.ProcessToken(c) if err != nil { - return c.JSON(fiber.Map{ - "status": "error", - "message": "Autorização inválida", - }) + return fiber.ErrUnauthorized } database.DB.Where("id = ?", userCode).First(&user) if user.Id == 0 { - return c.JSON(fiber.Map{ - "status": "error", - "message": "(A2) - Autorização inválida", - }) + return fiber.ErrUnauthorized } return c.JSON(fiber.Map{ - "status": "success", "user": user, }) } @@ -159,23 +132,16 @@ func GetAllUsers(c *fiber.Ctx) error { _, err := utils.ProcessToken(c) if err != nil { - return c.JSON(fiber.Map{ - "status": "error", - "message": "Autorização inválida", - }) + return fiber.ErrUnauthorized } database.DB.Find(&users) if len(users) == 0 { - return c.JSON(fiber.Map{ - "status": "error", - "message": "Nenhum usuário encontrado", - }) + return fiber.ErrNotFound } return c.JSON(fiber.Map{ - "status": "success", "users": users, }) } diff --git a/controllers/eventController.go b/controllers/eventController.go index e7442a0..c2ef121 100644 --- a/controllers/eventController.go +++ b/controllers/eventController.go @@ -17,46 +17,31 @@ func AddEvent(c *fiber.Ctx) error { _, err := utils.ProcessToken(c) if err != nil { - return c.JSON(fiber.Map{ - "status": "error", - "message": "Sem autorização", - }) + return fiber.ErrUnauthorized } if err := c.BodyParser(&data); err != nil { - return err + return fiber.ErrBadRequest } u, err := strconv.Atoi(data["user"]) if err != nil { - return c.JSON(fiber.Map{ - "status": "error", - "message": "Requisição inválida (usuário não informado)", - }) + return fiber.ErrBadRequest } user := dbaccess.GetUserById(u) if user.Id == 0 { - return c.JSON(fiber.Map{ - "status": "error", - "message": "Requisição inválida (usuário inexistente)", - }) + return fiber.ErrBadRequest } dateformat := "02/01/2006 15:04" if data["startDt"] == "" { - return c.JSON(fiber.Map{ - "status": "error", - "message": "Requisição inválida (sem data)", - }) + return fiber.ErrBadRequest } else { startdt, err = time.Parse(dateformat, data["startDt"]) if err != nil { - return c.JSON(fiber.Map{ - "status": "error", - "message": "Requisição inválida (data inválida)", - }) + return fiber.ErrBadRequest } } @@ -72,14 +57,10 @@ func AddEvent(c *fiber.Ctx) error { database.DB.Create(&event) if event.Id == 0 { - return c.JSON(fiber.Map{ - "status": "error", - "message": "(E1) - Não foi possível criar o evento", - }) + return fiber.ErrNotAcceptable } return c.JSON(fiber.Map{ - "status": "success", "event": event, }) } diff --git a/controllers/versionController.go b/controllers/versionController.go index e210c08..3027d3c 100644 --- a/controllers/versionController.go +++ b/controllers/versionController.go @@ -4,6 +4,6 @@ import "github.com/gofiber/fiber/v2" func Version(c *fiber.Ctx) error { return c.JSON(fiber.Map{ - "version": "1.0.11", + "version": "1.0.12", }) } diff --git a/utils/utils.go b/utils/utils.go index 8c03267..8fe1f87 100644 --- a/utils/utils.go +++ b/utils/utils.go @@ -52,7 +52,6 @@ func ProcessToken(c *fiber.Ctx) (interface{}, error) { claims, ok := tk.Claims.(jwt.MapClaims) if ok && tk.Valid { - fmt.Printf("claims user: %v\n", claims["user"]) return claims["user"], nil }