Add command execution and cancellation logging#896
Conversation
Fixes utksh1#710 Added asyncio timeout exception handling, logging with context, and retry logic to prevent silent scan failures
|
Hi, I have implemented timeout exception handling, retry logic and logging for issue #710. The CI is failing on backend-lint and frontend-checks. I'm a beginner contributor and don't have local environment set up to fix linting issues. Could you please guide me on what specific changes are needed? I'm eager to learn and fix this! |
utksh1
left a comment
There was a problem hiding this comment.
Thanks for the PR. This cannot merge while required CI is failing.
Current failing checks include Fresh-clone smoke test (Ubuntu), backend-lint, and frontend-checks. Please fix those failures and keep the patch focused on command execution/cancellation logging so it can be reviewed cleanly.
utksh1
left a comment
There was a problem hiding this comment.
Rechecking after the latest indentation fix: this is still blocked.
Required checks are still failing, including Fresh-clone smoke test (Ubuntu), formatting-hygiene, and backend-lint. Please fix those failures before this command execution/cancellation logging change can merge.
Description
Added logging statements in BaseScanner to improve visibility of command execution and cancellation events.
Related Issues
Fixes #710
Type of Change
How Has This Been Tested?
Checklist