Syncronization interval now set on config save. (#3421)

mattermost.go

@@ -133,10 +133,6 @@ func main() {
 			complianceI.StartComplianceDailyJob()
 		}
 
-		if ldapI := einterfaces.GetLdapInterface(); ldapI != nil {
-			ldapI.StartLdapSyncJob()
-		}
-
 		// wait for kill signal before attempting to gracefully shutdown
 		// the running service
 		c := make(chan os.Signal)

model/job.go

@@ -28,8 +28,11 @@ func removeTaskByName(name string) {
 	delete(tasks, name)
 }
 
-func getTaskByName(name string) *ScheduledTask {
+func GetTaskByName(name string) *ScheduledTask {
-	return tasks[name]
+	if task, ok := tasks[name]; ok {
+		return task
+	}
+	return nil
 }
 
 func GetAllTasks() *map[string]*ScheduledTask {

model/ldap.go

@@ -5,4 +5,5 @@ package model
 
 const (
 	USER_AUTH_SERVICE_LDAP = "ldap"
+	LDAP_SYNC_TASK_NAME    = "LDAP Syncronization"
 )

utils/config.go

@@ -186,6 +186,12 @@ func LoadConfig(fileName string) {
 
 	Cfg = &config
 	ClientCfg = getClientConfig(Cfg)
+
+	// Actions that need to run every time the config is loaded
+	if ldapI := einterfaces.GetLdapInterface(); ldapI != nil {
+		// This restarts the job if nessisary (works for config reloads)
+		ldapI.StartLdapSyncJob()
+	}
 }
 
 func getClientConfig(c *model.Config) map[string]string {