fix: proper parsing of environment variables

go.mod

@@ -14,8 +14,6 @@ require (
 	github.com/rs/cors v1.7.0
 	github.com/rubenv/sql-migrate v0.0.0-20200402132117-435005d389bc
 	github.com/satori/go.uuid v1.2.0
-	github.com/smartystreets/goconvey v1.6.4 // indirect
 	github.com/t-tiger/gorm-bulk-insert v0.0.0-20191014134946-beb77b81825f
-	google.golang.org/appengine v1.6.5 // indirect
 	gopkg.in/ini.v1 v1.50.0
 )

main.go

@@ -40,20 +40,15 @@ func readConfig() *models.Config {
 		log.Fatal(err)
 	}
 
-	// TODO: Use jinzhu/configor or so
+	env := utils.LookupFatal("ENV")
-	env, _ := os.LookupEnv("ENV")
+	dbType := utils.LookupFatal("WAKAPI_DB_TYPE")
-	dbType, valid := os.LookupEnv("WAKAPI_DB_TYPE")
+	dbUser := utils.LookupFatal("WAKAPI_DB_USER")
-	dbUser, valid := os.LookupEnv("WAKAPI_DB_USER")
+	dbPassword := utils.LookupFatal("WAKAPI_DB_PASSWORD")
-	dbPassword, valid := os.LookupEnv("WAKAPI_DB_PASSWORD")
+	dbHost := utils.LookupFatal("WAKAPI_DB_HOST")
-	dbHost, valid := os.LookupEnv("WAKAPI_DB_HOST")
+	dbName := utils.LookupFatal("WAKAPI_DB_NAME")
-	dbName, valid := os.LookupEnv("WAKAPI_DB_NAME")
+	dbPortStr := utils.LookupFatal("WAKAPI_DB_PORT")
-	dbPortStr, valid := os.LookupEnv("WAKAPI_DB_PORT")
 	dbPort, err := strconv.Atoi(dbPortStr)
 
-	if !valid {
-		log.Fatal("Environment variables missing or invalid.")
-	}
-
 	cfg, err := ini.Load("config.ini")
 	if err != nil {
 		log.Fatalf("Fail to read file: %v", err)

utils/environment.go

@@ -0,0 +1,14 @@
+package utils
+
+import (
+	"log"
+	"os"
+)
+
+func LookupFatal(key string) string {
+	v, ok := os.LookupEnv(key)
+	if !ok {
+		log.Fatalf("missing env variable '%s'", key)
+	}
+	return v
+}

version.txt

@@ -1 +1 @@
-1.2.0
+1.2.1