csrf.php

.idea/dataSources.local.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <project version="4">
-  <component name="dataSourceStorageLocal" created-in="IU-253.32098.101">
+  <component name="dataSourceStorageLocal" created-in="IU-261.25134.95">
     <data-source name="articles" uuid="315cb5c9-2b0f-435b-b602-59823b160908">
       <database-info product="SQLite" version="3.51.1" jdbc-version="4.2" driver-name="SQLite JDBC" driver-version="3.51.1.0" dbms="SQLITE" exact-version="3.51.1" exact-driver-version="3.51">
         <identifier-quote-string>&quot;</identifier-quote-string>

content/search-results.php

@@ -137,17 +137,6 @@ $categories = [
             <p class="s-res-meta"><span id="s-res-result-count"><?php echo $totalResultsCount; ?></span> Treffer für Ihre Suchanfrage "<?php echo htmlspecialchars($query); ?>"</p>
         </div>
 
-        <p
-                id="results-loading"
-                class="results-loading"
-                role="status"
-                aria-live="polite"
-                hidden
-        >
-            <span class="results-spinner" aria-hidden="true"></span>
-            Suchergebnisse werden geladen …
-        </p>
-
         <!-- Ergebnisliste -->
         <div class="s-res-list">
             <?php if (!empty($results)): ?>

content/showArticle.php

@@ -457,18 +457,6 @@ if ($replyAuthor === null) {
                 <button type="submit" class="button">
                     Kommentar senden
                 </button>
-
-                <p
-                        id="comment-loading"
-                        class="comment-loading"
-                        role="status"
-                        aria-live="polite"
-                        hidden
-                >
-                    <span class="comment-spinner" aria-hidden="true"></span>
-                    Kommentar wird gesendet …
-                </p>
-
             </form>
         <?php else: ?>
             <div class="comment-login-hint">

content/showCategory.php

@@ -113,17 +113,6 @@ $resultCount = count($results);
             </p>
         </div>
 
-        <p
-                id="results-loading"
-                class="results-loading"
-                role="status"
-                aria-live="polite"
-                hidden
-        >
-            <span class="results-spinner" aria-hidden="true"></span>
-            Kategoriebeiträge werden geladen …
-        </p>
-
         <!-- Ergebnisliste -->
         <div class="s-res-list">
             <?php if (!empty($results)): ?>

css/main.css

@@ -225,34 +225,4 @@ h1 {
     text-align: center;
     text-decoration: none;
     box-sizing: border-box;
-}
-.comment-loading {
-    display: flex;
-    align-items: center;
-    gap: 8px;
-    margin-top: 10px;
-}
-
-.comment-loading[hidden] {
-    display: none;
-}
-
-.comment-spinner {
-    width: 16px;
-    height: 16px;
-    border: 2px solid #cbd5e1;
-    border-top-color: #1f2937;
-    border-radius: 50%;
-    animation: comment-spinner-rotation 0.8s linear infinite;
-}
-
-@keyframes comment-spinner-rotation {
-    to {
-        transform: rotate(360deg);
-    }
-}
-
-#comment-form button[type="submit"]:disabled {
-    cursor: wait;
-    opacity: 0.65;
 }

css/search-results.css

@@ -271,33 +271,4 @@ CSS für die Suchergebnis-Seite
         align-items: flex-start;
     }
 }
-/* Ladeanzeige beim Nachladen von Such- und Kategorieergebnissen */
-.results-loading {
-    display: flex;
-    align-items: center;
-    gap: 8px;
-    margin: 15px 0;
-    color: #4a5568;
-    font-size: 0.95rem;
-}
-
-.results-loading[hidden] {
-    display: none;
-}
-
-.results-spinner {
-    width: 16px;
-    height: 16px;
-    flex-shrink: 0;
-    border: 2px solid #cbd5e1;
-    border-top-color: #3182ce;
-    border-radius: 50%;
-    animation: results-spinner-rotation 0.8s linear infinite;
-}
-
-@keyframes results-spinner-rotation {
-    to {
-        transform: rotate(360deg);
-    }
-}
 

includes/alertMessages.php

@@ -89,6 +89,12 @@
         Es ist ein Datenbankfehler aufgetreten. Bitte versuche es erneut.
     </p>
 <?php endif; ?>
+<?php if (isset($_SESSION["message"]) && $_SESSION["message"] == "invalid_csrf_token"): ?>
+    <p class="alert-message is-error">
+        Deine Sitzung ist abgelaufen oder die Anfrage konnte nicht überprüft werden.
+        Bitte lade die Seite neu und versuche es erneut.
+    </p>
+<?php endif; ?>
 <?php
 unset($_SESSION["message"]);
-?>
+?>

includes/csrf.php

@@ -0,0 +1,88 @@
+<?php
+/**
+ * CSRF-Schutz nach dem Synchronizer-Token-Pattern.
+ *
+ * Pro Session wird ein einziges, zufälliges Token erzeugt,
+ * das in jedem Formular als verstecktes Feld mitgeschickt und bei jeder
+ * zustandsändernden Anfrage serverseitig mit dem Session-Token verglichen
+ * wird.
+ *
+ * @author Niklas Ortmann
+ */
+
+if (session_status() === PHP_SESSION_NONE) {
+    session_start();
+}
+
+/**
+ * Gibt das aktuelle CSRF-Token der Session zurück.
+ *
+ * Existiert noch kein Token, wird eines erzeugt und in der Session
+ * gespeichert.
+ *
+ * @return string Das CSRF-Token
+ */
+function csrf_token(): string
+{
+    if (empty($_SESSION["csrf_token"]) || !is_string($_SESSION["csrf_token"])) {
+        $_SESSION["csrf_token"] = bin2hex(random_bytes(32));
+    }
+
+    return $_SESSION["csrf_token"];
+}
+
+/**
+ * Gibt ein verstecktes Formularfeld mit dem aktuellen CSRF-Token aus.
+ *
+ * Wird in jedem Formular benötigt, das eine zustandsändernde
+ * Aktion auslöst.
+ *
+ * @return void
+ */
+function csrf_field(): void
+{
+    echo '<input type="hidden" name="csrf_token" value="'
+        . htmlspecialchars(csrf_token())
+        . '">';
+}
+
+/**
+ * Prüft, ob das per POST gesendete CSRF-Token zum Session-Token passt.
+ *
+ * Der Vergleich erfolgt zeitkonstant über hash_equals(), um
+ * Timing-Angriffe auf den Vergleich selbst auszuschließen.
+ *
+ * @return bool true, wenn das Token gültig ist
+ */
+function csrf_verify(): bool
+{
+    $sentToken = $_POST["csrf_token"] ?? "";
+    $sessionToken = $_SESSION["csrf_token"] ?? "";
+
+    if (!is_string($sentToken) || $sentToken === "" || $sessionToken === "") {
+        return false;
+    }
+
+    return hash_equals($sessionToken, $sentToken);
+}
+
+/**
+ * Bricht die Anfrage ab und leitet mit einer Fehlermeldung um,
+ * wenn das mitgesendete CSRF-Token ungültig oder nicht vorhanden ist.
+ *
+ * Muss am Anfang jeder zustandsändernden POST-Aktion aufgerufen werden,
+ * bevor irgendeine Änderung an Daten vorgenommen wird.
+ *
+ * @param string $redirectTo Ziel-URL, zu der bei ungültigem Token
+ *                           weitergeleitet wird
+ * @return void
+ */
+function csrf_require_valid(string $redirectTo = "index.php"): void
+{
+    if (!csrf_verify()) {
+        http_response_code(403);
+        $_SESSION["message"] = "invalid_csrf_token";
+        header("Location: " . $redirectTo);
+        exit();
+    }
+}

index.php

@@ -2,6 +2,7 @@
 if (session_status() === PHP_SESSION_NONE) {
     session_start();
 }
+include_once "includes/csrf.php";
 include_once "php/controller/index-controller.php";
 ?>
 <!DOCTYPE html>

js/comments.js

@@ -11,19 +11,7 @@ document.addEventListener("DOMContentLoaded", function () {
     const parentCommentInput = document.getElementById("parent-comment-id");
     const replyInfo = document.getElementById("reply-info");
 
-    const submitButton = form
+    if (!form || !commentsList || !commentContent || !parentCommentInput) {
-        ? form.querySelector('button[type="submit"]')
-        : null;
-
-    const loadingMessage = document.getElementById("comment-loading");
-
-    if (
-        !form
-        || !commentsList
-        || !commentContent
-        || !parentCommentInput
-        || !submitButton
-    ) {
         return;
     }
 
@@ -65,23 +53,6 @@ document.addEventListener("DOMContentLoaded", function () {
     form.addEventListener("submit", function (event) {
         event.preventDefault();
 
-        /*
-         * Verhindert einen erneuten Submit, während der vorherige
-         * Kommentar noch gespeichert wird.
-         */
-        if (submitButton.disabled) {
-            return;
-        }
-
-        const originalButtonText = submitButton.textContent;
-
-        submitButton.disabled = true;
-        submitButton.textContent = "Wird gesendet …";
-
-        if (loadingMessage) {
-            loadingMessage.hidden = false;
-        }
-
         const formData = new FormData(form);
         const parentCommentId = parentCommentInput.value;
 
@@ -298,14 +269,6 @@ document.addEventListener("DOMContentLoaded", function () {
             .catch(function (error) {
                 console.error(error);
                 alert("Kommentar konnte nicht gesendet werden.");
-            })
-            .finally(function () {
-                submitButton.disabled = false;
-                submitButton.textContent = originalButtonText;
-
-                if (loadingMessage) {
-                    loadingMessage.hidden = true;
-                }
             });
     });
 

js/search-results.js

@@ -25,7 +25,6 @@
 
     let listContainer, sortRadios, categorySelect, limitSelect;
     let prevBtn, nextBtn, numbersContainer, jsNav, resultCountEl;
-    let loadingElement;
 
     function init() {
         listContainer = document.querySelector('.s-res-list');
@@ -39,41 +38,23 @@
         numbersContainer = document.getElementById('dynamic-page-numbers');
         jsNav = document.getElementById('js-page-navigation');
         resultCountEl = document.getElementById('s-res-result-count');
-        loadingElement = document.getElementById('results-loading');
 
         const checkedRadio = document.querySelector('.sort-radio:checked');
         state.sort = checkedRadio ? checkedRadio.value : 'alphabet';
         state.category = categorySelect ? categorySelect.value : 'all';
         state.itemsPerPage = limitSelect ? (parseInt(limitSelect.value, 10) || 10) : 10;
 
-        if (loadingElement) {
-            loadingElement.hidden = false;
-        }
-
         fetchFullDataset()
             .then(function (data) {
                 state.allItems = data.results || [];
                 attachEvents();
                 render();
-
+                if (jsNav) jsNav.style.display = 'flex';
-                if (jsNav) {
-                    jsNav.style.display = 'flex';
-                }
             })
             .catch(function (err) {
-                /*
+                // JSON-Endpoint nicht erreichbar: die serverseitig gerenderte
-                 * Falls das Nachladen nicht funktioniert, bleibt die bereits
+                // (No-JS-)Ansicht bleibt sichtbar und funktioniert weiter.
-                 * serverseitig ausgegebene Ergebnisliste sichtbar.
+                console.error('Suchergebnisse konnten nicht nachgeladen werden:', err);
-                 */
-                console.error(
-                    'Suchergebnisse konnten nicht nachgeladen werden:',
-                    err
-                );
-            })
-            .finally(function () {
-                if (loadingElement) {
-                    loadingElement.hidden = true;
-                }
             });
     }
 

js/showCategory.js

@@ -21,7 +21,6 @@
 
     let listContainer, sortRadios, searchInput, limitSelect;
     let prevBtn, nextBtn, numbersContainer, jsNav, resultCountEl;
-    let loadingElement;
     let searchDebounceTimer;
 
     function init() {
@@ -36,41 +35,23 @@
         numbersContainer = document.getElementById('dynamic-page-numbers');
         jsNav = document.getElementById('js-page-navigation');
         resultCountEl = document.getElementById('s-res-result-count');
-        loadingElement = document.getElementById('results-loading');
 
         const checkedRadio = document.querySelector('.sort-radio:checked');
         state.sort = checkedRadio ? checkedRadio.value : 'alphabet';
         state.query = searchInput ? searchInput.value : '';
         state.itemsPerPage = limitSelect ? (parseInt(limitSelect.value, 10) || 10) : 10;
 
-        if (loadingElement) {
-            loadingElement.hidden = false;
-        }
-
         fetchFullDataset()
             .then(function (data) {
                 state.allItems = data.results || [];
                 attachEvents();
                 render();
-
+                if (jsNav) jsNav.style.display = 'flex';
-                if (jsNav) {
-                    jsNav.style.display = 'flex';
-                }
             })
             .catch(function (err) {
-                /*
+                // JSON-Endpoint nicht erreichbar: die serverseitig gerenderte
-                 * Falls das Nachladen nicht funktioniert, bleibt die bereits
+                // (No-JS-)Ansicht bleibt sichtbar und funktioniert weiter.
-                 * serverseitig ausgegebene Kategorieansicht sichtbar.
+                console.error('Kategorie-Beiträge konnten nicht nachgeladen werden:', err);
-                 */
-                console.error(
-                    'Kategorie-Beiträge konnten nicht nachgeladen werden:',
-                    err
-                );
-            })
-            .finally(function () {
-                if (loadingElement) {
-                    loadingElement.hidden = true;
-                }
             });
     }
 

php/model/DatabaseArticleManager.php

@@ -310,20 +310,21 @@ class DatabaseArticleManager implements ArticleManagerDAO {
 
         try {
             $db = $this->getConnection();
-            
+
             $sql = "SELECT id, title, content, author, category, tags, created 
-                FROM articles 
+                    FROM articles 
-                WHERE title LIKE :keyword 
+                    WHERE title LIKE :keyword 
-                OR content LIKE :keyword
+                       OR content LIKE :keyword
-                OR tags LIKE :keyword;";
+                       OR tags LIKE :keyword";
 
             $command = $db->prepare($sql);
             if (!$command) {
                 throw new InternalServerErrorException("internal_error");
             }
 
-            // Wildcards für die SQL-Suche hinzufügen
+            // Wildcards für die Suche hinzufügen
             $searchParam = '%' . $cleankeyword . '%';
+
             $success = $command->execute([
                 ":keyword" => $searchParam
             ]);
@@ -336,11 +337,10 @@ class DatabaseArticleManager implements ArticleManagerDAO {
             $filteredArticles = [];
 
             foreach ($rows as $row) {
-                $articleId = intval($row['id']);
+                $likes = $this->getLikesForArticle(intval($row['id']));
-                $likes = $this->getLikesForArticle($articleId);
 
                 $filteredArticles[] = new Article(
-                    $articleId,
+                    intval($row['id']),
                     $row['title'] ?? '',
                     $row['content'] ?? '',
                     $row['author'] ?? '',
@@ -358,7 +358,6 @@ class DatabaseArticleManager implements ArticleManagerDAO {
         }
     }
 
-
     /**
      * Holt alle User-IDs, die einen bestimmten Beitrag geliked haben.
      *