blu3-bird/Shopping-on-tuesday

[Bug] Incorrect exception handling in admin routes

Aberta

#47 aberto em 10 de dez. de 2025

Ver no GitHub
 (0 comentário) (0 reação) (0 responsável)CSS (16 forks)auto 404
frontendhelp wantedinvalid🐕bug🐭good first issue🫎enhancement

Métricas do repositório

Stars
 (5 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

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

Guia do colaborador

Direção de pesquisa
Substitua as classes de exceção redundantes (DataError, OperationalError, IntegrityError) apenas por SQLAlchemyError e adicione registro de erros usando logging.exception(e) em cada bloco catch em app/admin/routes.py.
Pilha de tecnologia
pythonflask
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
PythonFlaskSQLAlchemy
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80