From ad96a4341c80bb6d4f7d087f0caf62212be63c0c Mon Sep 17 00:00:00 2001
From: Kailash Nadh <kailash@nadh.in>
Date: Thu, 1 Nov 2018 21:59:38 +0530
Subject: [PATCH] Add simple sanitsation to subscriber name

---
 subscribers.go | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/subscribers.go b/subscribers.go
index fd4a91f..4fb9f3d 100644
--- a/subscribers.go
+++ b/subscribers.go
@@ -178,14 +178,12 @@ func handleCreateSubscriber(c echo.Context) error {
 		return echo.NewHTTPError(http.StatusBadRequest, err.Error())
 	}
 
-	req.Email = strings.ToLower(strings.TrimSpace(req.Email))
-
 	// Insert and read ID.
 	var newID int
 	err := app.Queries.UpsertSubscriber.Get(&newID,
 		uuid.NewV4(),
-		req.Email,
-		req.Name,
+		strings.ToLower(strings.TrimSpace(req.Email)),
+		strings.TrimSpace(req.Name),
 		req.Status,
 		req.Attribs,
 		true,
@@ -227,11 +225,9 @@ func handleUpdateSubscriber(c echo.Context) error {
 		return echo.NewHTTPError(http.StatusBadRequest, "Invalid length for `name`.")
 	}
 
-	req.Email = strings.ToLower(strings.TrimSpace(req.Email))
-
 	_, err := app.Queries.UpdateSubscriber.Exec(req.ID,
-		req.Email,
-		req.Name,
+		strings.ToLower(strings.TrimSpace(req.Email)),
+		strings.TrimSpace(req.Name),
 		req.Status,
 		req.Attribs,
 		req.Lists)