aboutcode-org / aboutcode-org/vulnerablecode

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

Đang mở
#2,070 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
Ngôn ngữ chính
Python
Star
702
Fork
328
Merge trung bình
3 ngày 8 giờ
Pull request đã merge (30 ngày)
3

Mô tả

## 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()}"
)

Hướng dẫn đóng góp

Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.