Sicherheitslücke
Hallo @Janno_Liivak,
Vielen Dank für dieses nützliche Plugin! Ich habe einige kritische Sicherheitslücken gefunden, die Aufmerksamkeit erfordern:
Probleme
- Keine Autorisierungsprüfung – Jeder Benutzer kann jedes Thema als verkauft/gekauft/getauscht markieren
- Fehlende Backend-Validierung – Controller überprüfen nicht:
- Plugin aktiviert (
topic_trade_buttons_enabled) - Schaltflächen für Kategorien aktiviert (
enable_*_button) - Nur das Frontend prüft diese Einstellungen (unsicher)
- Plugin aktiviert (
- Keine Eingabevalidierung – Der Parameter
topic_idwird nicht validiert - Keine Aktion nach Erstellung – Operationen werden nicht protokolliert, keine Aufzeichnung darüber, wer Aktionen ausgeführt hat
Auswirkungen
- Nicht autorisierte Themenmanipulation
- Umgehung von Plugin-/Kategorieeinstellungen durch direkte API-Aufrufe
- Kein Audit-Trail darüber, wer Handelsaktionen ausgeführt hat