Skip to content

Include network interface info in SCAN event#3211

Open
liquidsec wants to merge 1 commit into
devfrom
scan-network-info
Open

Include network interface info in SCAN event#3211
liquidsec wants to merge 1 commit into
devfrom
scan-network-info

Conversation

@liquidsec

Copy link
Copy Markdown
Collaborator

Summary

  • Adds a network key to the SCAN event data containing hostname, primary outbound IP, all network interfaces (IPv4/IPv6 with netmasks), and default routes
  • Uses psutil (existing dep) for interface enumeration and /proc/net/route for default gateway detection (graceful fallback on non-Linux)
  • Enables correlating scan activity back to a specific agent/IP

Closes #2137

@github-actions

Copy link
Copy Markdown
Contributor

🚀 Performance Benchmark Report

⚠️ No current benchmark data available

This might be because:

  • Benchmarks failed to run
  • No benchmark tests found
  • Dependencies missing

@codecov

codecov Bot commented Jun 18, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 87.09677% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 90%. Comparing base (82df903) to head (3d029bb).
⚠️ Report is 3 commits behind head on dev.

Files with missing lines Patch % Lines
bbot/scanner/scanner.py 85% 8 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff          @@
##             dev   #3211   +/-   ##
=====================================
- Coverage     90%     90%   -0%     
=====================================
  Files        453     453           
  Lines      46101   46163   +62     
=====================================
+ Hits       41216   41265   +49     
- Misses      4885    4898   +13     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant