From 8fadea44debd00c1babd12bff9eb2dd0a5d99370 Mon Sep 17 00:00:00 2001
From: =Corey Hulen <corey@hulen.com>
Date: Tue, 29 Sep 2015 14:17:16 -0700
Subject: PLT-404 adding basic error checking to config file.

---
 api/admin.go | 11 ++---------
 1 file changed, 2 insertions(+), 9 deletions(-)

(limited to 'api/admin.go')

diff --git a/api/admin.go b/api/admin.go
index 568d8f6e8..3ef8c12a8 100644
--- a/api/admin.go
+++ b/api/admin.go
@@ -82,18 +82,11 @@ func saveConfig(c *Context, w http.ResponseWriter, r *http.Request) {
 		return
 	}
 
-	if len(cfg.ServiceSettings.ListenAddress) == 0 {
-		c.SetInvalidParam("saveConfig", "config")
-		return
-	}
-
-	if cfg.TeamSettings.MaxUsersPerTeam == 0 {
-		c.SetInvalidParam("saveConfig", "config")
+	if err := cfg.IsValid(); err != nil {
+		c.Err = err
 		return
 	}
 
-	// TODO run some cleanup validators
-
 	utils.SaveConfig(utils.CfgFileName, cfg)
 	utils.LoadConfig(utils.CfgFileName)
 	json := utils.Cfg.ToJson()
-- 
cgit v1.2.3-1-g7c22