aboutcode-org / aboutcode-org/vulnerablecode

fix: Replace bare [except] (vulnerabilities/tests/test_bare_except_fix.py) with explicit `Exception` in import_runner.py

Aperta
#2,070 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
Python
Stelle
702
Fork
328
Merge medio
3g 8h
PR unite (30g)
3

Descrizione

## Summary

The file [vulnerabilities/import_runner.py] contains a bare `except:` clause at line 222 which violates PEP 8 (E722) and can lead to unexpected behavior.

## Problem

Bare `except:` clauses catch **all** exceptions, including:
- `SystemExit` - prevents clean program termination
- `KeyboardInterrupt` - prevents Ctrl+C from working
- `GeneratorExit` - can break generator cleanup

This is almost never the intended behavior and can mask critical errors or make debugging difficult.

## Location

**File:** [vulnerabilities/import_runner.py](cci:7://file:///D:/Projects2.0/Gsoc/vulnerablecode/vulnerabilities/import_runner.py:0:0-0:0)
**Line:** 222
**Current code:**
```python
except:
logger.error(
f"Failed to create VulnerabilitySeverity for: {severity} with error:\n{traceback_format_exc()}"
)

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.