[BUG] Replace faulty cleanup go-routine with cron job

Author: hypherionmc

cleanup.go

@@ -2,13 +2,15 @@ package main
 
 import (
 	"fmt"
+	"github.com/robfig/cron/v3"
 	"os"
 	"path/filepath"
 	"time"
 )
 
 func cleanupCache() {
 	now := time.Now()
+	fmt.Printf("Starting cleanup at %v\n", now)
 
 	err := filepath.Walk(skinCacheDir, func(path string, info os.FileInfo, err error) error {
 		if err != nil {
@@ -35,14 +37,15 @@ func cleanupCache() {
 	}
 }
 
-func startCleanupRoutine() {
-	ticker := time.NewTicker(2 * time.Hour)
-	defer ticker.Stop()
+func startCleanupRoutine() *cron.Cron {
+	c := cron.New()
 
-	for {
-		select {
-		case <-ticker.C:
-			cleanupCache()
-		}
+	_, err := c.AddFunc("0 */2 * * *", cleanupCache)
+	if err != nil {
+		fmt.Printf("Failed to start cleanup routine: %v\n", err)
+		return nil
 	}
+
+	c.Start()
+	return c
 }

go.mod

@@ -14,6 +14,7 @@ require (
 require (
 	github.com/cespare/xxhash/v2 v2.3.0 // indirect
 	github.com/dgryski/go-rendezvous v0.0.0-20200823014737-9f7001d12a5f // indirect
+	github.com/robfig/cron/v3 v3.0.1 // indirect
 	golang.org/x/net v0.38.0 // indirect
 	golang.org/x/sys v0.31.0 // indirect
 )

go.sum

@@ -20,6 +20,8 @@ github.com/onsi/ginkgo v1.16.5 h1:8xi0RTUf59SOSfEtZMvwTvXYMzG4gV23XVHOZiXNtnE=
 github.com/onsi/ginkgo v1.16.5/go.mod h1:+E8gABHa3K6zRBolWtd+ROzc/U5bkGt0FwiG042wbpU=
 github.com/onsi/gomega v1.18.1 h1:M1GfJqGRrBrrGGsbxzV5dqM2U2ApXefZCQpkukxYRLE=
 github.com/onsi/gomega v1.18.1/go.mod h1:0q+aL8jAiMXy9hbwj2mr5GziHiwhAIQpFmmtT5hitRs=
+github.com/robfig/cron/v3 v3.0.1 h1:WdRxkvbJztn8LMz/QEvLN5sBU+xKpSqwwUO1Pjr4qDs=
+github.com/robfig/cron/v3 v3.0.1/go.mod h1:eQICP3HwyT7UooqI/z+Ov+PtYAWygg1TEWWzGIFLtro=
 golang.org/x/net v0.38.0 h1:vRMAPTMaeGqVhG5QyLJHqNDwecKTomGeqbnfZyKlBI8=
 golang.org/x/net v0.38.0/go.mod h1:ivrbrMbzFq5J41QOQh0siUuly180yBYtLp+CKbEaFx8=
 golang.org/x/sys v0.31.0 h1:ioabZlmFYtWhL+TRYpcnNlLwhyxaM9kWTDEmfnprqik=

main.go

@@ -12,8 +12,17 @@ const renderDir = "cached_skins/render"
 func main() {
 	os.MkdirAll(skinCacheDir, os.ModePerm)
 	os.MkdirAll(renderDir, os.ModePerm)
+
+	// Cleanup Logic
+	// Initial Cleanup
 	cleanupCache()
-	go startCleanupRoutine()
+
+	// Cron
+	cleanup := startCleanupRoutine()
+	if cleanup != nil {
+		defer cleanup.Stop()
+	}
+
 	initRedis()
 	initHttp()
 }