Issue #146

CHANGE:
  - config.yaml Made a note about how whitelisting defaults to false.
  - config/config.go We now explicitly parse a whitelist value from a string

config.yaml

@@ -7,8 +7,8 @@ RedisServer: localhost
 RedisPort: 6379
 # Whitelist indicates that torrent whitelisting is activated. When torrent
 # whitelisting is activated, only pre-approved torrents can be served by
-# Notorious
-Whitelist: false
+# Notorious. If whitelist is unable to be parsed, we will default to `false`
+whitelist: false
 # UseEnvVariables allows environmental variables to be used instead of
 # expliclitly defined config variables.
 UseEnvVariables: true

config/config.go

@@ -2,6 +2,7 @@ package config
 
 import (
 	"github.com/spf13/viper"
+        "strconv"
 )
 
 // ConfigStruct holds the values that our config file holds
@@ -34,6 +35,10 @@ func LoadConfig() ConfigStruct {
                 viper.BindEnv("mysqluser")
         }
 
+        whitelist, err := strconv.ParseBool(viper.Get("whitelist").(string))
+        if err != nil {
+                whitelist = false
+        }
 
 	if viper.Get("MySQLPass").(string) != "" {
 		return ConfigStruct{
@@ -42,7 +47,7 @@ func LoadConfig() ConfigStruct {
 			viper.Get("mysqluser").(string),
 			viper.Get("mysqlpass").(string),
 			viper.Get("mysqldb").(string),
-			viper.Get("whitelist").(bool),
+			whitelist,
 		}
 	} else {
 		return ConfigStruct{
@@ -51,7 +56,7 @@ func LoadConfig() ConfigStruct {
 			viper.Get("mysqluser").(string),
 			"",
 			viper.Get("mysqldb").(string),
-			viper.Get("whitelist").(bool),
+			whitelist,
 		}
 	}