remove comment

.env

@@ -1,6 +1,8 @@
-ENV=            #empty|development|debug
+#empty|development|debug
+ENV=            
 PHOTO_DIR=images
 HOST=0.0.0.0
 PORT=8080
-INTERVAL_DEFAULT=10
+INTERVAL_DEFAULT=120
-LOG_PATH= #path for logger files default .
+#path for logger files default {executable name}.log
+LOG_PATH=

.gitea/workflows/build.yml

@@ -98,6 +98,7 @@ jobs:
           # 2. Create Directory Structure
           mkdir -p $BUILD_DIR/usr/bin
           mkdir -p $BUILD_DIR/usr/share/$PKG_NAME
+          mkdir -p $BUILD_DIR/usr/share/$PKG_NAME/data
           mkdir -p $BUILD_DIR/etc/systemd/system
           mkdir -p $BUILD_DIR/DEBIAN
 
@@ -113,23 +114,14 @@ jobs:
             echo "PORT=8080" > $BUILD_DIR/usr/share/$PKG_NAME/.env.template
           fi
 
-          # 5. Create the systemd Service File
+          # 5. Create a Template Autostart File (Instead of Systemd)
-          cat <<EOF > $BUILD_DIR/etc/systemd/system/$PKG_NAME.service
+          mkdir -p $BUILD_DIR/usr/share/$PKG_NAME/setup
-          [Unit]
+          cat <<'EOF' > $BUILD_DIR/usr/share/$PKG_NAME/setup/$PKG_NAME.desktop
-          Description=Slideshow Application Service
+          [Desktop Entry]
-          After=graphical.target
+          Type=Application
-
+          Name=Slideshow App
-          [Service]
+          Exec=sh -c 'i=0; while [ $i -lt 30 ]; do if [ "$(ls -A /media/$USER 2>/dev/null)" ] && ping -c 1 -W 1 8.8.8.8 >/dev/null 2>&1; then break; fi; i=$((i+1)); sleep 1; done; cd /usr/share/slideshowapp && /usr/bin/slideshowapp'
-          Type=simple
+          Terminal=false
-          Environment=DISPLAY=:0
-          Environment=XAUTHORITY=/home/youruser/.Xauthority
-          ExecStart=/usr/bin/$PKG_NAME
-          WorkingDirectory=/usr/share/$PKG_NAME
-          Restart=always
-          User=root
-
-          [Install]
-          WantedBy=multi-user.target
           EOF
           
           # 6. Create Control File
@@ -139,28 +131,64 @@ jobs:
           Section: utils
           Priority: optional
           Architecture: $ARCH
-          Maintainer: Your Name <you@example.com>
+          Maintainer: Adrian Zuercher <zuercher@tecamino.ch>
           Description: Slideshow Application
           EOF
 
-          # 7. Optional: Add post-install script to enable service automatically
+          # 7. Add post-install script for Autostart and Permissions
-          echo "create service"
+          cat <<'EOF' > $BUILD_DIR/DEBIAN/postinst
-          cat <<EOF > $BUILD_DIR/DEBIAN/postinst
           #!/bin/sh
-          # Check if .env exists in the target share folder
+          set -e  # Exit on error
-          if [ ! -f "/usr/share/$PKG_NAME/.env" ]; then
+
-              echo "Creating new .env from template..."
+          # 1. Detect the real user
-              cp /usr/share/$PKG_NAME/.env.template /usr/share/$PKG_NAME/.env
+          # We use 'who' and 'awk' as a fallback because $SUDO_USER is sometimes empty in Gitea/CI environments
-          else
+          REAL_USER=${SUDO_USER:-$(who | awk '{print $1}' | head -n1)}
-              echo ".env already exists, skipping template copy to preserve settings."
+
+          # If still empty, default to 'mst' or 'pi' (adjust to your primary user)
+          if [ -z "$REAL_USER" ] || [ "$REAL_USER" = "root" ]; then
+              REAL_USER="mst" 
           fi
 
-          echo "reload"
+          USER_HOME=$(getent passwd "$REAL_USER" | cut -d: -f6)
-          systemctl daemon-reload
+
-          echo "enable service"
+          echo "Setting permissions for /usr/share/slideshowapp..."
-          systemctl enable $PKG_NAME.service
+          chown -R "$REAL_USER:$REAL_USER" /usr/share/slideshowapp
-          echo "start service"
+          chmod -R 755 /usr/share/slideshowapp
-          systemctl start $PKG_NAME.service
+
+          echo "Post-install: Target user is $REAL_USER"
+          echo "Post-install: Target home is $USER_HOME"
+
+          # 2. Setup .env from template
+          if [ ! -f "/usr/share/slideshowapp/.env" ]; then
+              echo "Creating .env from template..."
+              cp /usr/share/slideshowapp/env.template /usr/share/slideshowapp/.env || true
+          fi
+
+          # 3. Setup Autostart
+          # We use -p to ensure parent directories exist and set ownership immediately
+          AUTOSTART_DIR="$USER_HOME/.config/autostart"
+
+          if [ -d "$USER_HOME" ]; then
+              echo "Creating autostart directory at $AUTOSTART_DIR"
+              mkdir -p "$AUTOSTART_DIR"
+              cp /usr/share/slideshowapp/setup/slideshowapp.desktop "$AUTOSTART_DIR/"
+              
+              # Critical: Change ownership of the folder and the file
+              chown -R "$REAL_USER:$REAL_USER" "$USER_HOME/.config"
+              chmod 644 "$AUTOSTART_DIR/slideshowapp.desktop"
+          else
+              echo "ERROR: Home directory $USER_HOME not found. Autostart not configured."
+          fi
+
+          # 4. Fix Chromium Profile Permissions
+          if [ -d "$USER_HOME/.config/chromium" ]; then
+              echo "Cleaning up Chromium locks for $REAL_USER..."
+              chown -R "$REAL_USER:$REAL_USER" "$USER_HOME/.config/chromium"
+              rm -f "$USER_HOME/.config/chromium/SingletonLock"
+              rm -f "$USER_HOME/.config/chromium/SingletonCookie"
+          fi
+
+          echo "Installation complete."
           EOF
           chmod 555 $BUILD_DIR/DEBIAN/postinst
 

.gitignore

@@ -1,3 +1,4 @@
 images/
 slideshowApp
 schedule.json
+*.log

go.mod

@@ -3,7 +3,7 @@ module slideshowApp
 go 1.25.4
 
 require (
-	gitea.tecamino.com/paadi/tecamino-logger v1.0.0
+	gitea.tecamino.com/paadi/tecamino-logger v1.0.1
 	github.com/gorilla/mux v1.8.1
 	github.com/gorilla/websocket v1.5.3
 	github.com/joho/godotenv v1.5.1

go.sum

@@ -1,5 +1,5 @@
-gitea.tecamino.com/paadi/tecamino-logger v1.0.0 h1:Quk0UJerOCy7xbHWYOuBXk0FQIgx4arjVDMt2ahvaO0=
+gitea.tecamino.com/paadi/tecamino-logger v1.0.1 h1:NCF/foyuf0wVKLkyy2y1FYvHo9pVaUuHc1GSU5bNVbc=
-gitea.tecamino.com/paadi/tecamino-logger v1.0.0/go.mod h1:FkzRTldUBBOd/iy2upycArDftSZ5trbsX5Ira5OzJgM=
+gitea.tecamino.com/paadi/tecamino-logger v1.0.1/go.mod h1:FkzRTldUBBOd/iy2upycArDftSZ5trbsX5Ira5OzJgM=
 github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c=
 github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
 github.com/gorilla/mux v1.8.1 h1:TuBL49tXwgrFYWhqrNgrUNEY92u81SPhu7sTdzQEiWY=

handlers/files.go

@@ -73,7 +73,7 @@ func DeleteHandler(w http.ResponseWriter, r *http.Request) {
 
 	// 2. Delete each file
 	for _, name := range filenames {
-		fullPath := filepath.Join(uploadDir, filepath.Base(name)) // Base() for security
+		fullPath := filepath.Join(uploadDir, filepath.Base(name))
 		os.Remove(fullPath)
 	}
 

handlers/infos.go

@@ -47,7 +47,7 @@ func InfoHandler(w http.ResponseWriter, r *http.Request) {
 	data := map[string]string{
 		"ip":    GetLocalIP(),
 		"port":  port,
-		"speed": speed,
+		"speed": GetInterval(),
 	}
 	json.NewEncoder(w).Encode(data)
 }

handlers/scheduler.go

@@ -1,26 +0,0 @@
-package handlers
-
-import (
-	"encoding/json"
-	"net/http"
-	"os"
-)
-
-type Schedule map[string]interface{}
-
-func SaveSchedule(w http.ResponseWriter, r *http.Request) {
-	var s Schedule
-	json.NewDecoder(r.Body).Decode(&s)
-	data, _ := json.Marshal(s)
-	os.WriteFile("schedule.json", data, 0644)
-	w.WriteHeader(http.StatusOK)
-}
-
-func GetSchedule(w http.ResponseWriter, r *http.Request) {
-	data, err := os.ReadFile("schedule.json")
-	if err != nil {
-		json.NewEncoder(w).Encode(Schedule{})
-		return
-	}
-	w.Write(data)
-}

handlers/settings.go

@@ -0,0 +1,81 @@
+package handlers
+
+import (
+	"encoding/json"
+	"fmt"
+	"net/http"
+	"os"
+	"path/filepath"
+	"runtime"
+	"slideshowApp/env"
+	"slideshowApp/utils"
+)
+
+var settingsFile = "settings.json"
+
+type Settings map[string]any
+
+func SaveSettings(w http.ResponseWriter, r *http.Request) {
+	var s Settings
+	if err := json.NewDecoder(r.Body).Decode(&s); err != nil {
+		fmt.Println(12, err)
+		utils.SendJSONError(w, "Invalid input", http.StatusBadRequest)
+		return
+	}
+
+	data, err := json.MarshalIndent(s, "", "  ")
+	if err != nil {
+		fmt.Println(13, err)
+
+		utils.SendJSONError(w, "Encoding failed", http.StatusInternalServerError)
+		return
+	}
+
+	err = os.WriteFile(getFilePath(), data, 0644)
+	if err != nil {
+		fmt.Println(14, err)
+
+		utils.SendJSONError(w, "write file failed", http.StatusInternalServerError)
+		return
+	}
+	w.WriteHeader(http.StatusOK)
+}
+
+func GetSettings(w http.ResponseWriter, r *http.Request) {
+	data, err := os.ReadFile(getFilePath())
+	if err != nil {
+		json.NewEncoder(w).Encode(Settings{"default_interval": GetInterval()})
+		return
+	}
+	w.Write(data)
+}
+
+func GetInterval() string {
+	data, err := os.ReadFile(getFilePath())
+	if err != nil {
+		interval := env.IntervalDefault.GetValue()
+		if interval == "" {
+			interval = "10"
+		}
+		return interval
+	}
+	var settings Settings
+	err = json.Unmarshal(data, &settings)
+	if err != nil {
+		return "10"
+	}
+
+	if interval, ok := settings["default_interval"]; ok {
+		return interval.(string)
+	}
+	return "10"
+}
+
+func getFilePath() string {
+	switch runtime.GOOS {
+	case "linux":
+		return filepath.Join(".", "data", settingsFile)
+	default:
+		return settingsFile
+	}
+}

main.go

@@ -18,6 +18,8 @@ var staticFolder = "./web/static/"
 
 func main() {
 
+	env.Load(".env")
+
 	logConfig := logging.DefaultConfig()
 	logConfig.Debug = env.Env.GetValue() == "debug"
 
@@ -26,9 +28,6 @@ func main() {
 		log.Fatal(err)
 	}
 
-	logger.Debug("main", "load enviroment variable")
-	env.Load(".env")
-
 	r := mux.NewRouter()
 
 	uploadFolder := env.PhotoDir.GetValue()
@@ -51,8 +50,8 @@ func main() {
 	r.HandleFunc("/ws", handlers.Websocket)
 	r.HandleFunc("/upload", handlers.UploadHandler).Methods("POST")
 	r.HandleFunc("/settings", func(w http.ResponseWriter, r *http.Request) { http.ServeFile(w, r, staticFolder+"settings.html") })
-	r.HandleFunc("/api/save-schedule", handlers.SaveSchedule).Methods("POST")
+	r.HandleFunc("/api/save-settings", handlers.SaveSettings).Methods("POST")
-	r.HandleFunc("/api/get-schedule", handlers.GetSchedule).Methods("GET")
+	r.HandleFunc("/api/get-settings", handlers.GetSettings).Methods("GET")
 
 	r.HandleFunc("/api/delete", handlers.DeleteHandler).Methods("POST")
 	r.HandleFunc("/manage", func(w http.ResponseWriter, r *http.Request) {

utils/http.go

@@ -0,0 +1,12 @@
+package utils
+
+import (
+	"encoding/json"
+	"net/http"
+)
+
+func SendJSONError(w http.ResponseWriter, message string, code int) {
+	w.Header().Set("Content-Type", "application/json")
+	w.WriteHeader(code)
+	json.NewEncoder(w).Encode(map[string]string{"error": message})
+}

web/static/frame.html

@@ -33,7 +33,7 @@
 
         <button id="shuffleBtn" onclick="toggleShuffle()" class="text-xs uppercase font-bold text-gray-400 hover:text-white transition-colors flex items-center gap-2">
             <svg class="w-4 h-4" fill="none" stroke="currentColor" viewBox="0 0 24 24"><path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M4 4v5h.582m15.356 2A8.001 8.001 0 004.582 9m0 0H9m11 11v-5h-.581m0 0a8.003 8.003 0 01-15.357-2m15.357 2H15" /></svg>
-            Shuffle: <span id="shuffleStatus">OFF</span>
+            Shuffle: <span id="shuffleStatus">ON</span>
         </button>
 
         <button onclick="toggleFullScreen()" class="text-sm hover:text-blue-400 transition-colors">Full Screen</button>
@@ -58,6 +58,18 @@
         const speedInput = document.getElementById('speed');
         const shuffleStatus = document.getElementById('shuffleStatus');
 
+        async function initializeSlideshow() {
+            // 1. Get the default interval from API
+            const defaultInterval = await checkDefaultInterval();
+            speedInput.value = defaultInterval;
+            
+            // 2. Setup QR Code
+            setupQRCode();
+            
+            // 3. Load Images (which eventually calls start())
+            loadImages();
+        }
+
         // Fisher-Yates Shuffle Algorithm
         function shuffleArray(array) {
             for (let i = array.length - 1; i > 0; i--) {
@@ -75,10 +87,23 @@
             index = 0;
         }
 
-        async function checkSchedule() {
+        async function checkDefaultInterval() {
             try {
-                const res = await fetch('/api/get-schedule');
+                const res = await fetch('/api/get-settings'); 
-                const schedule = await res.json();
+                const settings = await res.json();
+                
+                // Return the saved interval, or fallback to 10 if it's missing
+                return settings.default_interval || "120"; 
+            } catch (err) {
+                console.error("Failed to fetch default interval:", err);
+                return "120"; 
+            }
+        }
+
+        async function checkSettings() {
+            try {
+                const res = await fetch('/api/get-settings');
+                const settings = await res.json();
                 
                 const now = new Date();
                 const day = now.toLocaleDateString('en-US', { weekday: 'long' });
@@ -86,9 +111,9 @@
                 const currentTime = now.getHours().toString().padStart(2, '0') + ":" + 
                                 now.getMinutes().toString().padStart(2, '0');
 
-                const isActive = schedule[day + "_active"];
+                const isActive = settings[day + "_active"];
-                const start = schedule[day + "_start"];
+                const start = settings[day + "_start"];
-                const end = schedule[day + "_end"];
+                const end = settings[day + "_end"];
 
                 // Check if we should be "OFF"
                 if (isActive && (currentTime < start || currentTime > end)) {
@@ -131,7 +156,7 @@
         async function showNext() {
             if (playlist.length === 0) return;
 
-            const isRunning = await checkSchedule();
+            const isRunning = await checkSettings();
             if (!isRunning) return;
             
             viewer.classList.replace('fade-in', 'fade-out');
@@ -205,8 +230,7 @@
         speedInput.addEventListener('change', start);
         
         // Initial setup
-        setupQRCode();
+        initializeSlideshow();
-        loadImages();
     </script>
 </body>
 </html>

web/static/settings.html

@@ -8,11 +8,24 @@
 <body class="bg-gray-50 min-h-screen p-6">
     <div class="max-w-2xl mx-auto bg-white rounded-xl shadow-lg p-8 border border-gray-100">
         <div class="flex justify-between items-center mb-8">
-            <h1 class="text-2xl font-bold text-gray-800">Weekly Scheduler</h1>
+            <h1 class="text-2xl font-bold text-gray-800">Slideshow Settings</h1>
             <a href="/" class="text-blue-600 hover:underline text-sm">Back</a>
         </div>
 
-        <form id="scheduleForm" class="space-y-4">
+        <form id="scheduleForm" class="space-y-8">
+            <div class="space-y-4">
+                <h2 class="text-sm font-semibold text-gray-400 uppercase tracking-wider">General Settings</h2>
+                <div class="flex items-center justify-between p-4 bg-gray-50 rounded-lg border border-gray-100">
+                    <label class="text-gray-700 font-medium">Default Interval (seconds)</label>
+                    <input type="number" name="default_interval" min="1" value="10" 
+                           class="w-24 bg-white border rounded-md p-2 text-center focus:ring-2 focus:ring-blue-500 outline-none">
+                </div>
+            </div>
+
+            <hr class="border-gray-100">
+
+            <div class="space-y-4">
+                <h2 class="text-sm font-semibold text-gray-400 uppercase tracking-wider">Weekly Scheduler</h2>
                 <table class="w-full text-left">
                     <thead>
                         <tr class="text-gray-400 text-xs uppercase">
@@ -25,7 +38,11 @@
                     <tbody class="text-gray-600 italic text-sm">
                         </tbody>
                 </table>
-            <button type="submit" class="w-full bg-blue-600 text-white py-3 rounded-lg font-bold shadow-md hover:bg-blue-700 transition">Save Schedule</button>
+            </div>
+
+            <button type="submit" class="w-full bg-blue-600 text-white py-3 rounded-lg font-bold shadow-md hover:bg-blue-700 transition">
+                Save All Settings
+            </button>
         </form>
     </div>
 
@@ -47,32 +64,57 @@
 
         // Load existing settings
         async function loadSettings() {
-            const res = await fetch('/api/get-schedule');
+            try {
+                const res = await fetch('/api/get-settings');
                 const data = await res.json();
+                
                 Object.keys(data).forEach(key => {
                     const el = document.querySelector(`[name="${key}"]`);
+                    if (el) {
                         if (el.type === 'checkbox') el.checked = data[key];
                         else el.value = data[key];
+                    }
                 });
+            } catch (err) {
+                console.error("Failed to load settings:", err);
+            }
         }
 
         document.getElementById('scheduleForm').onsubmit = async (e) => {
             e.preventDefault();
             const formData = new FormData(e.target);
             const obj = {};
-            formData.forEach((value, key) => {
-                // Handle checkbox value manually
-                if (key.includes('_active')) obj[key] = true;
-                else obj[key] = value;
-            });
-            // Ensure unchecked boxes are false
-            days.forEach(day => { if(!obj[day+'_active']) obj[day+'_active'] = false; });
 
-            await fetch('/api/save-schedule', {
+            formData.forEach((value, key) => {
+                // Convert number fields to integers
+                if (key === 'default_interval') {
+                    obj[key] = parseInt(value);
+                } 
+                // Handle checkbox presence
+                else if (key.includes('_active')) {
+                    obj[key] = true;
+                } 
+                else {
+                    obj[key] = value;
+                }
+            });
+
+            // Ensure unchecked boxes are sent as false
+            days.forEach(day => { 
+                if (!obj[day + '_active']) obj[day + '_active'] = false; 
+            });
+
+            const res = await fetch('/api/save-settings', {
                 method: 'POST',
+                headers: { 'Content-Type': 'application/json' },
                 body: JSON.stringify(obj)
             });
-            alert("Schedule Saved!");
+
+            if (res.ok) {
+                alert("Settings Saved Successfully!");
+            } else {
+                alert("Error saving settings.");
+            }
         };
 
         loadSettings();