Ich möchte vorschlagen, eine maximale Verschachtelungstiefe für die eingebaute [details]-Funktion hinzuzufügen oder eine Seiteneinstellung einzuführen, mit der Administratoren dieses Limit konfigurieren können.
Das Problem
Kürzlich hat ein Benutzer auf meinem Discourse-Forum einen Beitrag erstellt, der 24 Ebenen verschachtelter [details]-Abschnitte enthielt.
Dies führte zu mehreren Problemen:
- Browser-Performance: Benutzer meldeten, dass das Aufklappen mehrerer Ebenen dazu führte, dass Firefox extrem langsam oder nicht mehr ansprechend wurde.
- Timeouts der Markdown-Endpunkte: Der Zugriff auf den
.md-Endpunkt des Themas wurde extrem langsam. Wir beobachteten Anfragen, die 19,8 Sekunden und 31,1 Sekunden in Anspruch nahmen. - Serverfehler: Der Markdown-Endpunkt gab manchmal eine „Oops“-Fehlerseite zurück. Die Discourse-Logs zeigten während der Markdown-Konvertierung auch Pitchfork-Worker-Timeout-Warnungen.
- Potenzielle Ressourcenerschöpfung: Da Suchmaschinen und KI-Crawler häufig
.md-Endpunkte anfordern, könnte ein stark verschachtelter Beitrag wiederholt aufwendige Verarbeitungsschritte auslösen und Serverressourcen verbrauchen.
Nach dem Entfernen des problematischen Beitrags gab der Markdown-Endpunkt HTTP 200 zurück und reagierte in etwa 1,7 Sekunden.
Das betroffene Thema war:
https://meta.appinn.net/t/topic/87672
(Die problematische Antwort wurde bereits entfernt.)
Vorschlag zur Verbesserung
Ich denke, es wäre nützlich, entweder:
- Die Verschachtelung von
[details]auf eine angemessene Tiefe zu begrenzen, z. B. 2 oder 3 Ebenen. - Eine Seiteneinstellung hinzuzufügen, wie z. B.
details_max_nesting_depth, die es Administratoren ermöglicht, die maximale Verschachtelungstiefe zu konfigurieren.
Zur Abwärtskompatibilität könnte die Einstellung standardmäßig auf unbegrenzt gesetzt sein, während Administratoren bei Bedarf ein Limit durchsetzen können.
Wenn Benutzer das konfigurierte Limit überschreiten, könnte Discourse den Beitrag mit einer klaren Validierungsnachricht ablehnen.
Temporäre Lösung: ein Plugin
Ich habe ein kleines Plugin erstellt, das die Verschachtelung von [details] auf maximal 2 Ebenen begrenzt:
GitHub - scavin/discourse-details-depth-limit · GitHub
Das Plugin validiert Beiträge serverseitig und lehnt Beiträge ab, die mehr als zwei Ebenen verschachtelter aufklappbarer Abschnitte enthalten.
Ich habe es erfolgreich in meiner lokalen Discourse-Entwicklungsumgebung getestet.
Für Administratoren, die ähnliche Probleme erleben, kann dieses Plugin als vorübergehende Lösung dienen, bis eine offizielle Lösung verfügbar ist.
Ich bin der Meinung, dass ein konfigurierbares Limit im eingebauten Details-Plugin eine nützliche Schutzmaßnahme gegen versehentliche oder übermäßige Verschachtelung wäre.