Boutons de trading

Greffon
121

:warning: vulnérabilité de sécurité :warning:

Salut @Janno_Liivak,

Merci pour ce plugin utile ! J’ai trouvé des vulnérabilités de sécurité critiques qui nécessitent une attention particulière :

Problèmes

  1. Aucune vérification d’autorisation - N’importe quel utilisateur peut marquer n’importe quel sujet comme vendu/acheté/échangé
  2. Validation backend manquante - Les contrôleurs ne vérifient pas :
    • Plugin activé (topic_trade_buttons_enabled)
    • Boutons de catégorie activés (enable_*_button)
    • Seul le frontend vérifie ces paramètres (dangereux)
  3. Aucune validation d’entrée - Le paramètre topic_id n’est pas validé
  4. Aucune action post-création - Les opérations ne sont pas enregistrées, aucun enregistrement de qui a effectué les actions

Impact

  • Manipulation non autorisée des sujets
  • Contournement des paramètres du plugin/de la catégorie via des appels API directs
  • Aucune piste d’audit de qui a effectué les actions de transaction
4 « J'aime »