Merge pull request #12 from Panonim/unstable

Author: Panonim

README.md

@@ -22,9 +22,9 @@ services:
       PASSWORD: ${QB_PASSWORD}
       BASE_URL: ${QB_URL}
       AUTH_TOKEN: ${AUTH_TOKEN}
-      LISTEN_PORT: "9911"
-      RATE_LIMIT: "10"  # API requests per minute (default: 10)
     restart: unless-stopped
+    volumes:
+      - ./logs:/app/logs # Optional
 ```
 `.env`
 ```env
@@ -33,6 +33,14 @@ QB_PASSWORD=
 QB_URL=http://IP:PORT
 AUTH_TOKEN=REPLACEME
 ```
+`Optional Environment Variables`
+```yaml
+RATE_LIMIT: "10"          # API requests per minute (default: 10)
+LOG_RETENTION_DAYS: 3     # 0 for purge-on-restart
+LOG_DIR: /app/logs        # Where docker should save your logs
+DEBUG: true               # Logs level
+LISTEN_PORT: "9911".      # Port on which qBW should listen internally
+```
 
 ## Environment Variables
 
@@ -41,21 +49,21 @@ You **must** provide these in a `.env` file or your environment:
 * `QB_URL` — base URL of your qBittorrent Web UI (e.g., `http://localhost:8080`)
 * `QB_USERNAME` — your qBittorrent username
 * `QB_PASSWORD` — your qBittorrent password
-* `AUTH_TOKEN` — Bearer token required to access the `/qb/torrents` endpoint
+* `AUTH_TOKEN` — This is not a qBittorrent token. It’s a bearer token used by qBWrapper to control access (like a password) to the /qb/torrents endpoint.
 
 ## Glance
 ```yaml
 - type: custom-api
   title: qBittorrent
   cache: 15m
   options:
-    always-show-stats: false 
+    always-show-stats: true 
   subrequests:
     info:
-      url: "http://${QB_URL}/qb/torrents"
+      url: "http://${QBW_URL}/qb/torrents"
       method: GET
       headers:
-        Authorization: "Bearer ${AUTH_TOKEN}"  # your token
+        Authorization: "Bearer ${AUTH_TOKEN}"  # your QBW token
   template: |
     {{ $info := .Subrequest "info" }}
     {{ $torrents := $info.JSON.Array "" }}
@@ -161,8 +169,8 @@ You **must** provide these in a `.env` file or your environment:
 
 ### Glance env
 You can put this app in the same place as glance and the same .env file, but in case you are using it alone please put this in your `.env`.
-* `QB_URL` — base URL of your qBittorrent Web UI (e.g., `http://localhost:8080`)
-* `AUTH_TOKEN` — Bearer token required to access the `/qb/torrents` endpoin
+* `QBW_URL` — base URL of your qBittorrent Wrapper (e.g., `http://localhost:9911`)
+* `AUTH_TOKEN` — This is not a qBittorrent token. It’s a bearer token used by qBWrapper to control access (like a password) to the /qb/torrents endpoint.
 
 ## What you get in the response
 
@@ -183,4 +191,3 @@ Each torrent object includes:
 * The cache is locked for concurrency safety.
 * If the qBittorrent login fails, the app exits.
 * If you hit the endpoint without a valid token, you get a 401 Unauthorized.
-* The app uses standard Go HTTP server and `github.com/joho/godotenv` for env loading.

app/main.go

@@ -13,6 +13,7 @@ import (
         "strings"
         "sync"
         "time"
+        "path/filepath"
 )
 
 var (
@@ -351,16 +352,71 @@ func toFloat(i interface{}) float64 {
         return 0
 }
 
+func cleanOldLogs(logDir string, retentionDays int) {
+        if retentionDays == 0 {
+                log.Println("LOG_RETENTION_DAYS=0; purging all logs")
+        } else {
+                log.Printf("Cleaning logs older than %d days in %s\n", retentionDays, logDir)
+        }
+
+        now := time.Now()
+        _ = filepath.Walk(logDir, func(path string, info os.FileInfo, err error) error {
+                if err != nil || info.IsDir() {
+                        return nil
+                }
+
+                if !strings.HasSuffix(info.Name(), ".log") {
+                        return nil
+                }
+
+                age := now.Sub(info.ModTime())
+                if retentionDays == 0 || age > (time.Duration(retentionDays)*24*time.Hour) {
+                        log.Printf("Removing old log: %s (age: %v)", path, age.Round(time.Second))
+                        os.Remove(path)
+                }
+                return nil
+        })
+}
+
 func main() {
         baseURL = strings.TrimRight(os.Getenv("BASE_URL"), "/")
         username = os.Getenv("USERNAME")
         password = os.Getenv("PASSWORD")
         authToken = os.Getenv("AUTH_TOKEN")
         debug = os.Getenv("DEBUG") == "true"
 
+        logDir := os.Getenv("LOG_DIR")
+        if logDir == "" {
+                logDir = "./logs"
+        }
+        if err := os.MkdirAll(logDir, 0755); err != nil {
+                fmt.Printf("Failed to create log directory: %v\n", err)
+                os.Exit(1)
+        }
+
+        logFileName := filepath.Join(logDir, "app-"+time.Now().Format("2006-01-02_15-04-05")+".log")
+        logFile, err := os.OpenFile(logFileName, os.O_CREATE|os.O_WRONLY|os.O_APPEND, 0644)
+        if err != nil {
+                fmt.Printf("Failed to open log file: %v\n", err)
+                os.Exit(1)
+        }
+
+        log.SetOutput(io.MultiWriter(os.Stdout, logFile))
+
         if baseURL == "" || username == "" || password == "" || authToken == "" {
                 log.Fatal("Missing required environment variables")
         }
+
+        retention := 3
+        if val := os.Getenv("LOG_RETENTION_DAYS"); val != "" {
+                if parsed, err := strconv.Atoi(val); err == nil && parsed >= 0 {
+                        retention = parsed
+                } else {
+                        log.Printf("Invalid LOG_RETENTION_DAYS: %s, defaulting to %d", val, retention)
+                }
+        }
+
+        cleanOldLogs(logDir, retention)
         initRateLimit()
 
         if err := login(); err != nil {
@@ -371,6 +427,7 @@ func main() {
         if port == "" {
                 port = "9911"
         }
+
         http.HandleFunc("/qb/torrents", authMiddleware(rateLimitMiddleware(torrentsHandler)))
         log.Printf("Listening on :%s\n", port)
         log.Fatal(http.ListenAndServe(":"+port, nil))