Merge pull request 'dev' (#4) from dev into main

Reviewed-on: #4

srv/poe_manager/app.py

@@ -4,7 +4,6 @@ from flask_login import LoginManager, login_user, login_required, logout_user, U
 from flask_bcrypt import Bcrypt
 from cryptography.fernet import Fernet
 from datetime import datetime
-from collections import defaultdict
 import sqlite3, glob, os, re
 
 app = Flask(__name__)
@@ -119,13 +118,6 @@ def index():
     c.execute("SELECT mac, name, is_active FROM devices ORDER BY name ASC")
     devices = c.fetchall()
 
-    devices = sorted(devices, key=lambda d: d[1][0].upper())
-
-    grouped_devices = defaultdict(list)
-    for d in devices:
-        first_letter = d[1][:2].upper()
-        grouped_devices[first_letter].append(d)
-
     # Intervall aus DB laden
     c.execute("SELECT value FROM settings WHERE key='interval'")
     row = c.fetchone()
@@ -159,7 +151,7 @@ def index():
             else:
                 status_dict[dev[0]] = "unbekannt"
 
-    return render_template("index.html", grouped_devices=grouped_devices, devices=devices, status=status_dict, last_seen=last_seen_dict, interval=interval)
+    return render_template("index.html", devices=devices, status=status_dict, last_seen=last_seen_dict, interval=interval)
 
 @app.route("/settings", methods=["GET", "POST"])
 @login_required

srv/poe_manager/static/css/style.css

@@ -87,33 +87,25 @@ pre {
 
 #log-container {
     position: relative;
-    height: calc(100vh - 252px); /* Füllt die Seite minus Header */
+    height: calc(100vh - 150px); /* Füllt die Seite minus Header */
-    padding: 1rem; /* optional, Abstand innen */
-    box-sizing: border-box; /* damit Padding nicht die Höhe sprengt */
-    display: flex;
-    flex-direction: column;
 }
 
 #log-box {
-    flex: 1 1 auto; /* füllt den Container, berücksichtigt Header/Padding */
+    height: 97%;
     overflow: auto;
     white-space: pre-wrap;
     font-family: monospace;
-    border: 1px solid #dee2e6;
+    border: 1px solid #dee2e6; /* Bootstrap-like border */
+    padding: 1rem;
     background-color: #f8f9fa;
-    padding: 0.5rem;
 }
 
 #refresh-timer {
     position: absolute;
-    bottom: 18px;
+    bottom: 10px;
-    right: 30px;
+    right: 10px;
     font-size: 0.9em;
     color: gray;
-    background-color: rgba(255,255,255,0.7); /* optional, besser lesbar */
-    padding: 2px 4px;
-    border-radius: 3px;
-    pointer-events: none; /* damit Scrollbar nicht blockiert wird */
 }
 
 /* Tabelle anpassen */

srv/poe_manager/templates/index.html

@@ -7,34 +7,30 @@
         Nächste Prüfung in -- Sekunden
     </span>
 </h2>
-<div class="row g-3">
+<div class="row row-cols-1 row-cols-md-6 g-3">
-    {% for letter, group in grouped_devices.items() %}
+    {% for d in devices %}
-    <div class="row g-3">
+    <div class="col">
-        {% for d in group %}
+        <div class="card text-center p-2"
-        <div class="col-6 col-md-4 col-lg-3 col-xl-2">
+     {% if last_seen.get(d[0]) %}
-            <div class="card text-center p-2"
+         title="{{ last_seen[d[0]] }}"
-                {% if last_seen.get(d[0]) %}
+     {% elif status[d[0]] == 'offline' %}
-                    title="{{ last_seen[d[0]] }}"
+         title="Noch nie online"
-                {% elif status[d[0]] == 'offline' %}
+     {% endif %}>
-                    title="Noch nie online"
+            <div class="card-header">{{ d[1] }}</div>
-                {% endif %}>
+            <div class="card-body">
-                <div class="card-header">{{ d[1] }}</div>
+                <span class="fw-bold" style="color: 
-                <div class="card-body">
+                    {% if d[2] == 0 %}gray
-                    <span class="fw-bold" style="color: 
+                    {% elif status[d[0]] == 'online' %}green
-                        {% if d[2] == 0 %}gray
+                    {% else %}red
-                        {% elif status[d[0]] == 'online' %}green
+                    {% endif %};">
-                        {% else %}red
+                    {% if d[2] == 0 %}
-                        {% endif %};">
+                        Deaktiviert
-                        {% if d[2] == 0 %}
+                    {% else %}
-                            Deaktiviert
+                        {% if status[d[0]] %}{{ status[d[0]]|capitalize }}{% else %}Unbekannt{% endif %}
-                        {% else %}
+                    {% endif %}
-                            {% if status[d[0]] %}{{ status[d[0]]|capitalize }}{% else %}Unbekannt{% endif %}
+                </span>
-                        {% endif %}
-                    </span>
-                </div>
             </div>
         </div>
-        {% endfor %}
     </div>
     {% endfor %}
 </div>

usr/local/bin/custom/poe.sh

@@ -87,45 +87,23 @@ EOF
 
 echo "" > "$LOGFILE"
 
-MAX_PARALLEL=10  # maximal gleichzeitig laufende Geräte
-
 while true; do
     echo "--------------------------------------------------------------------" >> "$LOGFILE"
     python3 /srv/poe_manager/generate_ips.py | while IFS=: read -r rpi_ip dev_name switch_ip switch_hostname switch_port switch_user switch_pass; do
-        # Funktion für ein Gerät
+        if ping -c 1 -W 2 "$rpi_ip" &> /dev/null; then
-        check_device() {
+            echo "$(date '+%Y-%m-%d %H:%M:%S') $dev_name ist erreichbar!" >> "$LOGFILE"
-            local rpi_ip="$1"
+        else
-            local dev_name="$2"
+            echo "$(date '+%Y-%m-%d %H:%M:%S') $dev_name ist nicht erreichbar!" >> "$LOGFILE"
-            local switch_ip="$3"
-            local switch_hostname="$4"
-            local switch_port="$5"
-            local switch_user="$6"
-            local switch_pass="$7"
 
-            if ping -c 4 -W 1 "$rpi_ip" &> /dev/null; then
+            # Nur PoE neu starten, wenn Port vorhanden ist
-                echo "$(date '+%Y-%m-%d %H:%M:%S') $dev_name ist erreichbar!" >> "$LOGFILE"
+            if [ -n "$switch_port" ] && [ "$switch_port" != "None" ]; then
-            else
+                disable_poe "$switch_ip" "$switch_port" "$switch_user" "$switch_pass"
-                echo "$(date '+%Y-%m-%d %H:%M:%S') $dev_name ist nicht erreichbar!" >> "$LOGFILE"
+                echo "$(date '+%Y-%m-%d %H:%M:%S') $dev_name PoE auf Port $switch_port am Switch $switch_hostname deaktiviert." >> "$LOGFILE"
-
+                sleep 2
-                if [ -n "$switch_port" ] && [ "$switch_port" != "None" ]; then
+                enable_poe "$switch_ip" "$switch_port" "$switch_user" "$switch_pass"
-                    disable_poe "$switch_ip" "$switch_port" "$switch_user" "$switch_pass"
+                echo "$(date '+%Y-%m-%d %H:%M:%S') $dev_name PoE auf Port $switch_port am Switch $switch_hostname aktiviert." >> "$LOGFILE"
-                    echo "$(date '+%Y-%m-%d %H:%M:%S') $dev_name PoE auf Port $switch_port am Switch $switch_hostname deaktiviert." >> "$LOGFILE"
-                    sleep 2
-                    enable_poe "$switch_ip" "$switch_port" "$switch_user" "$switch_pass"
-                    echo "$(date '+%Y-%m-%d %H:%M:%S') $dev_name PoE auf Port $switch_port am Switch $switch_hostname aktiviert." >> "$LOGFILE"
-                fi
             fi
-        }
+        fi
-
-        # Job in Hintergrund starten
-        check_device "$rpi_ip" "$dev_name" "$switch_ip" "$switch_hostname" "$switch_port" "$switch_user" "$switch_pass" &
-        
-        # Limit auf MAX_PARALLEL Jobs
-        while [ "$(jobs -rp | wc -l)" -ge "$MAX_PARALLEL" ]; do
-            sleep 0.2
-        done
     done
-
-    wait  # alle Hintergrundjobs beenden, bevor sleep
     sleep "$SLEEP"
 done