fix: config for backend;

refactor: docker compose env variables

backend/cmd/main.go

@@ -14,7 +14,7 @@ import (
 
 func main() {
 
-	if err := config.LoadConfig(); err != nil {
+	if _, err := config.Load(); err != nil {
 		panic(err)
 	}
 

backend/config/config.go

@@ -2,84 +2,74 @@ package config
 
 import (
 	"fmt"
-	"log"
+	"strings"
-	"reflect"
 
 	"github.com/spf13/viper"
 )
 
 type Config struct {
-	Hostname 				string `mapstructure:"HOSTNAME"`
+	Hostname     string `mapstructure:"HOSTNAME"`
-	Port     				string `mapstructure:"PORT"`
+	Port         string `mapstructure:"PORT"`
-
+	DatabaseUrl  string `mapstructure:"POSTGRES_URL"`
-	DatabaseUrl 		string `mapstructure:"POSTGRES_URL"`
+	RedisUrl     string `mapstructure:"REDIS_URL"`
-	RedisUrl 				string `mapstructure:"REDIS_URL"`
+	MinioUrl     string `mapstructure:"MINIO_URL"`
-	MinioUrl 				string `mapstructure:"MINIO_URL"`
+	JwtAlgorithm string `mapstructure:"JWT_ALGORITHM"`
-
+	JwtSecret    string `mapstructure:"JWT_SECRET"`
-	JwtAlgorithm 		string `mapstructure:"JWT_ALGORITHM"`
+	JwtIssuer    string `mapstructure:"JWT_ISSUER"`
-	JwtSecret 			string `mapstructure:"JWT_SECRET"`
+	JwtAudience  string `mapstructure:"JWT_AUDIENCE"`
-	JwtIssuer 			string `mapstructure:"JWT_ISSUER"`
+	JwtExpAccess string `mapstructure:"JWT_EXP_ACCESS"`
-	JwtAudience 		string `mapstructure:"JWT_AUDIENCE"`
+	JwtExpRefresh string `mapstructure:"JWT_EXP_REFRESH"`
-	JwtExpAccess 		string `mapstructure:"JWT_EXP_ACCESS"`
+	Environment  string `mapstructure:"ENVIRONMENT"`
-	JwtExpRefresh 	string `mapstructure:"JWT_EXP_REFRESH"`
-
-	Environment 		string `mapstructure:"ENVIRONMENT"`
 }
 
-var config *Config
+func Load() (*Config, error) {
-
-func LoadConfig() error {
-
-	viper.AutomaticEnv()
-
-	viper.SetConfigName(".env")
-	viper.SetConfigType("env")
-	viper.AddConfigPath(".")
-
-	// Load .env file
-	if err := viper.ReadInConfig(); err != nil {
-		log.Printf("Error reading .env file, proceeding with environment variables. %s", err)
-	}
 
 	viper.SetDefault("HOSTNAME", "localhost")
 	viper.SetDefault("PORT", "8080")
-
 	viper.SetDefault("JWT_ALGORITHM", "HS256")
-	viper.SetDefault("JWT_SECRET", "default_jwt_secret_please_change") // TODO: remove and randomly generate
+	viper.SetDefault("JWT_SECRET", "default_jwt_secret_please_change")
 	viper.SetDefault("JWT_EXP_ACCESS", "5m")
 	viper.SetDefault("JWT_EXP_REFRESH", "1w")
 	viper.SetDefault("JWT_AUDIENCE", "easywish")
 	viper.SetDefault("JWT_ISSUER", "easywish")
-	
 	viper.SetDefault("ENVIRONMENT", "production")
 
-	if err := viper.Unmarshal(&config); err != nil {
+	viper.AutomaticEnv()
-		log.Fatalf("Unable to decode into struct, %v", err)
+
+	required := []string{
+		"POSTGRES_URL",
+		"REDIS_URL",
+		"MINIO_URL",
 	}
-
+	var missing []string
-	if err := validateConfig(); err != nil {
+	for _, key := range required {
-		return err
+		if !viper.IsSet(key) {
-	}
+			missing = append(missing, key)
-
-	return nil
-}
-
-func validateConfig() error {
-	v := reflect.ValueOf(*config)
-	t := v.Type()
-
-	for i := range v.NumField() {
-		field := v.Field(i)
-		fieldType := t.Field(i)
-
-		if field.Kind() == reflect.String && field.String() == "" {
-			return fmt.Errorf("Missing required configuration: %s", fieldType.Name)
 		}
 	}
+	if len(missing) > 0 {
+		return nil, fmt.Errorf("missing required environment variables: %s", strings.Join(missing, ", "))
+	}
 
-	return nil
+	var cfg Config
+	if err := viper.Unmarshal(&cfg); err != nil {
+		return nil, fmt.Errorf("failed to unmarshal config: %w", err)
+	}
+
+	config = &cfg
+	return &cfg, nil
 }
 
 func GetConfig() *Config {
+
+	if config == nil {
+		
+		if _, err := Load(); err != nil {
+			panic(err)
+		}
+	}
+
 	return config
 }
+
+var config *Config

docker-compose.yml

@@ -15,6 +15,10 @@ services:
       interval: 5s
       timeout: 10s
       retries: 3 
+    environment:
+      POSTGRES_URL: ${POSTGRES_URL}
+      REDIS_URL: ${REDIS_URL}
+      MINIO_URL: ${MINIO_URL}
     ports:
       - "8080:8080"
     networks: