Merge pull request #46 from Killerrekt/master

minor fixes like email will be converted to lowercase and team max size is now 5
CodeChefVIT · Mar 16, 2024 · 9242fa4 · 9242fa4
2 parents 1e0894b + 5d66bf4
commit 9242fa4
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/internal/controllers/auth_controller.go b/internal/controllers/auth_controller.go
@@ -7,6 +7,7 @@ import (
 	"net/http"
 	"os"
 	"strconv"
+	"strings"
 	"time"
 
 	"github.com/golang-jwt/jwt/v5"
@@ -37,6 +38,8 @@ func Login(ctx echo.Context) error {
 		})
 	}
 
+	payload.Email = strings.ToLower(payload.Email)
+
 	user, err := services.FindUserByEmail(payload.Email)
 	if err != nil {
 		if errors.Is(err, sql.ErrNoRows) {

diff --git a/internal/controllers/user_controller.go b/internal/controllers/user_controller.go
@@ -39,6 +39,8 @@ func CreateUser(ctx echo.Context) error {
 		})
 	}
 
+	payload.Email = strings.ToLower(payload.Email)
+
 	_, err := services.FindUserByEmail(payload.Email)
 	if err != nil && !errors.Is(err, sql.ErrNoRows) {
 		return ctx.JSON(http.StatusInternalServerError, map[string]string{