dbcli / dbcli/mssql-cli

Improve exception handling in jsonrpcclient

Offen
#350 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
area: engineering systems
Vorherrschende Sprache
Python
Sterne
1.4k
Forks
191
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

The `jsonrpcclient.py` file has two instances of generic exception handling. Generic exception handling breaks PEP 8 and can make debugging issues more difficult, especially since exceptions are suppressed.

Although we are logging exceptions, I recommend re-working this logic. If we choose to keep generic exception handling then we may at least consider simplifying instances of redundant code where repeat logic is used across different exceptions.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Öffne jsonrpcclient.py und identifiziere die beiden Stellen mit generischer Ausnahmebehandlung. Lies zuerst die umgebende Protokollierung und die wiederholte Ausnahmebehandlungslogik, und bestimme anschließend, wie die Behandlung überarbeitet oder vereinfacht werden sollte, ohne Fehler stillschweigend zu verschleiern. Erledigt ist es, wenn die Ausnahmebehandlungspfade einem festgelegten Ansatz folgen und die gemeldete redundante Behandlung nicht mehr enthalten.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
cli
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.