From 8033d052fec47aa5a9d3d31e534d4af8778f68b7 Mon Sep 17 00:00:00 2001
From: Jamie Curnow <jc@jc21.com>
Date: Tue, 10 Jan 2023 12:50:46 +1000
Subject: [PATCH] Trim whitespace from names of objects on save

---
 backend/internal/entity/certificate/model.go | 3 +++
 backend/internal/entity/setting/model.go     | 3 +++
 backend/internal/entity/upstream/model.go    | 4 ++++
 3 files changed, 10 insertions(+)

diff --git a/backend/internal/entity/certificate/model.go b/backend/internal/entity/certificate/model.go
index ac276fc9..7dec8f2e 100644
--- a/backend/internal/entity/certificate/model.go
+++ b/backend/internal/entity/certificate/model.go
@@ -100,6 +100,9 @@ func (m *Model) Save() error {
 
 	m.setDefaultStatus()
 
+	// ensure name is trimmed of whitespace
+	m.Name = strings.TrimSpace(m.Name)
+
 	if m.ID == 0 {
 		m.ID, err = Create(m)
 	} else {
diff --git a/backend/internal/entity/setting/model.go b/backend/internal/entity/setting/model.go
index e66d7276..05e9061f 100644
--- a/backend/internal/entity/setting/model.go
+++ b/backend/internal/entity/setting/model.go
@@ -55,6 +55,9 @@ func (m *Model) Touch(created bool) {
 func (m *Model) Save() error {
 	var err error
 
+	// ensure name is trimmed of whitespace
+	m.Name = strings.TrimSpace(m.Name)
+
 	if m.ID == 0 {
 		m.ID, err = Create(m)
 	} else {
diff --git a/backend/internal/entity/upstream/model.go b/backend/internal/entity/upstream/model.go
index 408a8787..d1d189aa 100644
--- a/backend/internal/entity/upstream/model.go
+++ b/backend/internal/entity/upstream/model.go
@@ -2,6 +2,7 @@ package upstream
 
 import (
 	"fmt"
+	"strings"
 	"time"
 
 	"npm/internal/database"
@@ -73,6 +74,9 @@ func (m *Model) Save(skipConfiguration bool) error {
 		return fmt.Errorf("User ID must be specified")
 	}
 
+	// ensure name is trimmed of whitespace
+	m.Name = strings.TrimSpace(m.Name)
+
 	if !skipConfiguration {
 		// Set this upstream as requiring reconfiguration
 		m.Status = status.StatusReady