Fix MaxTTL config check and missing exit on invalid config root-gg#342

server/cmd/root.go

@@ -97,6 +97,7 @@ func initConfig() {
 	config, err = common.LoadConfiguration(configPath)
 	if err != nil {
 		fmt.Printf("Unable to load config : %s\n", err)
+		os.Exit(1)
 	}
 }
 

server/common/config.go

@@ -183,7 +183,7 @@ func (config *Configuration) Initialize() (err error) {
 		}
 	}
 
-	if config.DefaultTTL > config.MaxTTL {
+	if config.MaxTTL > 0 && config.DefaultTTL > 0 && config.MaxTTL < config.DefaultTTL {
 		return fmt.Errorf("DefaultTTL should not be more than MaxTTL")
 	}
 

server/common/config_test.go

@@ -103,6 +103,15 @@ func TestInitializeInvalidDefaultTTL(t *testing.T) {
 	require.Error(t, err, "able to initialize invalid config")
 }
 
+func TestInitializeInfiniteMaxTTL(t *testing.T) {
+	config := NewConfiguration()
+	config.DefaultTTL = 10 * 86400
+	config.MaxTTL = -1
+
+	err := config.Initialize()
+	require.NoError(t, err, "unable to initialize valid config")
+}
+
 func TestDisableAutoClean(t *testing.T) {
 	config := NewConfiguration()
 	require.True(t, config.IsAutoClean(), "invalid auto clean status")