Skip to content

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

Closed
SarthakB11 wants to merge 1 commit into
Noethys:masterfrom
SarthakB11:fix-sql-injection
Closed

SarthakB11 wants to merge 1 commit into
Noethys:masterfrom
SarthakB11:fix-sql-injection

Conversation

@SarthakB11

Copy link
Copy Markdown

Corrections de vulnérabilités d'injection SQL

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

Modifications apportées

  1. Remplacement du formatage des chaînes par des requêtes paramétrées dans toutes les opérations de base de données
  2. Ajout d'une gestion des erreurs appropriée 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 d'un style de passage de paramètres cohérent

Améliorations de la sécurité

  • Les opérations de base de données utilisent désormais 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 de tuples de paramètres au lieu de listes pour les paramètres de requête

Tests

Ces modifications ont été testées avec les backends de bases de données SQLite et MySQL. Toutes les opérations de base de données continuent de fonctionner correctement, tout en offrant une sécurité renforcée contre les attaques par injection SQL.

[Translated to English]

SQL Injection Vulnerability Fixes

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

Changes Made

  1. Replaced string formatting 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.

…rmatting with parameterized queries - Add proper error handling - Improve code organization and readability
@SarthakB11 SarthakB11 changed the title Fix SQL injection vulnerabilities in GestionDB.py - Replace string fo… Fix SQL injection vulnerabilities in GestionDB.py Apr 14, 2025
@SarthakB11 SarthakB11 changed the title Fix SQL injection vulnerabilities in GestionDB.py Correction des vulnérabilités d'injection SQL dans GestionDB.py Apr 14, 2025
@Noethys

Noethys commented Feb 1, 2026

Copy link
Copy Markdown
Owner

Les corrections ont été intégrées manuellement. Merci pour cette contribution.

@Noethys Noethys closed this Feb 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants