blu3-bird/Shopping-on-tuesday

[Bug] Incorrect exception handling in admin routes

Aperta

#47 aperta il 10 dic 2025

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)CSS (16 fork)auto 404
frontendhelp wantedinvalid🐕bug🐭good first issue🫎enhancement

Metriche repository

Star
 (5 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

[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

Guida contributor

Direzione di ricerca
Sostituisci le classi di eccezione ridondanti (DataError, OperationalError, IntegrityError) solo con SQLAlchemyError e aggiungi la registrazione degli errori utilizzando logging.exception(e) in ogni blocco catch in app/admin/routes.py.
Tech stack
pythonflask
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonFlaskSQLAlchemy
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80