Skip to content

Add support for real-time scan monitoring via probe progress callbacks #147

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
MFormenti wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Add support for real-time scan monitoring via probe progress callbacks #147

MFormenti wants to merge 7 commits into from

Conversation

@MFormenti
Copy link

@MFormenti MFormenti commented Nov 19, 2025 •
edited
Loading

Introduces a callback mechanism to notify listeners when probes complete during scans, enabling partial result generation and progress tracking.

Added:

  • ProbeProgressCallback<ReportT, StateT> functional interface with:
    • onProbeCompleted() method invoked after each probe execution
    • noOp() factory method for default no-op behavior
  • Scanner.setProgressCallback() to register external callbacks
  • Callback invocation in ThreadedScanJobExecutor after probe results are merged
  • Scan target name added to probe execution log messages for better debugging

mattiaformenti added 4 commits November 14, 2025 10:24
WIP
a8964fb
WIP
33cc18a
Add scan target to probe execution log messages
5f01542 
- Include report.getRemoteName() in probe execution logs
- Format: [target] [X/Y] probe_name probe executed
- Helps identify which target is being scanned during bulk scans
Update unit tests
bf6a698
@MFormenti MFormenti changed the title Add probe progress callback support for real-time monitoring Add support for real-time scan monitoring via probe progress callbacks Nov 19, 2025
@ic0ns
Copy link
Contributor

ic0ns commented Nov 28, 2025

@mmaehren this looks good to me - what do you think?

@MFormenti MFormenti marked this pull request as draft December 3, 2025 09:26
@ic0ns
Copy link
Contributor

ic0ns commented Dec 3, 2025

put this on hold - will ping again once we need feedback again

@MFormenti MFormenti marked this pull request as ready for review December 8, 2025 06:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@juaninf juaninf juaninf left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@MFormenti @ic0ns @juaninf