blu3-bird/Shopping-on-tuesday

[Bug] Incorrect exception handling in admin routes

Offen

#47 geöffnet am 10.12.2025

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)CSS (16 Forks)auto 404
frontendhelp wantedinvalid🐕bug🐭good first issue🫎enhancement

Repository-Metriken

Stars
 (5 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

[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

Contributor Guide

Research-Richtung
Ersetzen Sie die redundanten Ausnahmeklassen (DataError, OperationalError, IntegrityError) durch nur SQLAlchemyError und fügen Sie in jedem Catch Block in app/admin/routes.py eine Fehlerprotokollierung mit logging.exception(e) hinzu.
Tech Stack
pythonflask
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
PythonFlaskSQLAlchemy
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80