blu3-bird/Shopping-on-tuesday

[Bug] Incorrect exception handling in admin routes

Ouverte

#47 ouverte le 10 déc. 2025

Voir sur GitHub
 (0 commentaire) (0 réaction) (0 personne assignée)CSS (16 forks)auto 404
frontendhelp wantedinvalid🐕bug🐭good first issue🫎enhancement

Métriques du dépôt

Stars
 (5 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

[Bug] Incorrect exception handling in admin routes

Problem

app/admin/routes.py - Exception handling issues:

except (DataError, SQLAlchemyError, OperationalError, IntegrityError) as e:

SQLAlchemyError is parent class, others are redundant

-Exception variable e unused (no logging) -Repeated in add_product, edit_product, delete_product

Fix Needed

-Use only SQLAlchemyError -Add error logging with exception details Labels: bug, error-handling

Guide contributeur

Direction de recherche
Remplacez les classes d'exception redondantes (DataError, OperationalError, IntegrityError) par uniquement SQLAlchemyError et ajoutez une journalisation des erreurs en utilisant logging.exception(e) dans chaque bloc catch dans app/admin/routes.py.
Stack technique
pythonflask
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PythonFlaskSQLAlchemy
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80