Skip to content

Correction des vulnérabilités d'injection SQL dans GestionDB.py #83

Closed

Description

@SarthakB11

Ce PR corrige plusieurs vulnérabilités d'injection SQL potentielles dans le fichier GestionDB.py en implémentant des requêtes paramétrées et en améliorant la sécurité globale de la base de données.

Modifications apportées

  1. Remplacement de la concaténation de chaînes par des requêtes paramétrées dans toutes les opérations de base de données
  2. Ajout d'une gestion appropriée des erreurs pour les opérations de base de données
  3. Amélioration de l'organisation et de la lisibilité du code
  4. Ajout de docstrings pour une meilleure documentation
  5. Utilisation cohérente du style de passage des paramètres

Améliorations de sécurité

  • Les opérations de base de données utilisent maintenant des requêtes paramétrées au lieu de la concaténation de chaînes
  • Ajout d'un échappement approprié pour les noms de bases de données et de tables
  • Amélioration de la gestion des erreurs pour éviter les fuites d'informations
  • Utilisation cohérente des tuples de paramètres au lieu des listes pour les paramètres de requête

Tests

Les modifications ont été testées avec les backends de base de données SQLite et MySQL. Toutes les opérations de base de données continuent de fonctionner comme prévu tout en offrant une meilleure sécurité contre les attaques par injection SQL.

[Translated to English]

This PR fixes several potential SQL injection vulnerabilities in the GestionDB.py file by implementing parameterized queries and improving overall database security.

Changes Made

  1. Replaced string concatenation with parameterized queries in all database operations
  2. Added proper error handling for database operations
  3. Improved code organization and readability
  4. Added docstrings for better documentation
  5. Used consistent parameter passing style

Security Improvements

  • Database operations now use parameterized queries instead of string concatenation
  • Added proper escaping for database and table names
  • Improved error handling to prevent information leakage
  • Consistent use of parameter tuples instead of lists for query parameters

Testing

The changes have been tested with both SQLite and MySQL database backends. All database operations continue to work as expected while providing improved security against SQL injection attacks.

Activity

  1. changed the title [-]Corriger les vulnérabilités d'injection SQL dans les opérations de base de données[/-] [+]Correction des vulnérabilités d'injection SQL dans GestionDB.py[/+] on Apr 14, 2025
  2. SarthakB11 commented on Apr 14, 2025

    @SarthakB11
    Author

    #84 : attached pr

  3. SarthakB11 commented on Jul 18, 2026

    @SarthakB11
    Author

    Looks like this was resolved with a manual fix per the maintainer's note on #84. Closing since the underlying issue is addressed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions