AOSSIE-Org / AOSSIE-Org/PictoPy

Logging: InterceptHandler preserves exc_info and stack_info (should be cleared before formatting)

Đang mở
#708 0 bình luận 0 reaction 1 người được giao Được @VasuS609 nhận Xem trên GitHub
backend bug
Ngôn ngữ chính
Python
Star
283
Fork
679
Merge trung bình
7 ngày 2 giờ
Pull request đã merge (30 ngày)
3

Mô tả

### Is there an existing issue for this?

- [x] I have searched the existing issues

### What happened?

The InterceptHandler in backend/app/logging/setup_logging.py calls
self.format(record) while record.exc_info and record.stack_info are still set.

Python’s Formatter.format() automatically appends traceback and stack output if
these fields are truthy.

This violates the project rule that exc_info and stack_info should not be
preserved when rerouting logs.

Fix proposal:
record.exc_info = None
record.stack_info = None
msg = self.format(record)

This should be applied around lines 238–252.

### Record

- [x] I agree to follow this project's Code of Conduct

### Checklist before Submitting

- [x] Have you updated docs for it?
- [ ] Have you added unit tests?
- [x] Have you made sure unit tests pass?
- [x] Have you made sure code formatting is correct?
- [x] Do Your changes passes all tests?

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

Mở hướng dẫn đóng góp

Đá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.