redis bugfix

Author: mstgnz

cmd/main.go

@@ -66,7 +66,7 @@ func main() {
 	statsService := service.NewStatsService()
 
 	// Initialize cache service
-	cacheService, err := service.NewCacheService(config.GetEnvOrDefault("REDIS_URL", "redis://localhost:6379"))
+	cacheService, err := service.NewCacheService()
 	if err != nil {
 		logger.Error().Err(err).Msg("Failed to initialize cache service, continuing without cache")
 		cacheService = nil

docker-compose.yml

@@ -60,7 +60,7 @@ services:
     volumes:
       - redis_data:/data
       - ./redis.conf:/usr/local/etc/redis/redis.conf
-    command: redis-server /usr/local/etc/redis/redis.conf --appendonly yes
+    command: redis-server /usr/local/etc/redis/redis.conf --appendonly no --protected-mode no
     networks:
       - cdn
 

handler/image.go

@@ -103,7 +103,7 @@ func NewImage(minioClient *minio.Client, awsService service.AwsService, imageSer
 	bp.Start()
 
 	// Initialize cache service
-	cacheService, err := service.NewCacheService("redis://localhost:6379")
+	cacheService, err := service.NewCacheService()
 	if err != nil {
 		log.Printf("Failed to initialize cache service: %v", err)
 	}

pkg/middleware/redis_storage.go

@@ -33,7 +33,7 @@ type RedisStorage struct {
 
 // NewRedisStorage creates a new Redis storage adapter
 func NewRedisStorage() (*RedisStorage, error) {
-	cache, err := service.NewCacheService("")
+	cache, err := service.NewCacheService()
 	if err != nil {
 		return nil, err
 	}

redis.conf

@@ -12,11 +12,13 @@ protected-mode no
 maxmemory 256mb
 maxmemory-policy allkeys-lru
 
-# Persistence
-appendonly yes
-appendfsync everysec
-auto-aof-rewrite-percentage 100
-auto-aof-rewrite-min-size 64mb
+# Persistence - Using RDB only instead of AOF
+appendonly no
+save 900 1
+save 300 10
+save 60 10000
+dbfilename dump.rdb
+dir /data
 
 # Performance tuning
 tcp-backlog 511

service/cache.go

@@ -29,10 +29,8 @@ type redisCache struct {
 	misses int64
 }
 
-func NewCacheService(redisURL string) (CacheService, error) {
-	if redisURL == "" {
-		redisURL = config.GetEnvOrDefault("REDIS_URL", "redis://localhost:6379")
-	}
+func NewCacheService() (CacheService, error) {
+	redisURL := config.GetEnvOrDefault("REDIS_URL", "redis://cdn-redis:6379")
 
 	opt, err := redis.ParseURL(redisURL)
 	if err != nil {

service/cache_test.go

@@ -6,7 +6,7 @@ import (
 )
 
 func TestCacheService(t *testing.T) {
-	cache, err := NewCacheService("redis://localhost:6379")
+	cache, err := NewCacheService()
 	if err != nil {
 		t.Fatalf("failed to create cache service: %v", err)
 	}