Skip to content

Cleanup scan, merge report stuff, hide report command#354

Merged
jdalton merged 3 commits intomainfrom
scan_v2
Mar 4, 2025
Merged

Cleanup scan, merge report stuff, hide report command#354
jdalton merged 3 commits intomainfrom
scan_v2

Conversation

@pvdz
Copy link
Contributor

@pvdz pvdz commented Mar 4, 2025

Deprecates and hides the socket report command, updates the socket scan command with a socket scan view command (supplementing the socket scan stream), updated some markdown/json flows.

  • The socket report commands are now hidden and marked as deprecated explicitly
  • The socket report view command idea is added to the scan suite
  • There's one feature that socket report create had that scan does not yet; using a file based config. We'll have to think about this a bit more before we codify that forever, but before the next version bump. Other than that they were equal already.
  • Added a socket scan view command, which kind of replaces the socket scan stream
    • stream does not convey the same message as view
    • The thing stream did is kept though; it'll stream the report to a file
    • Supports --markdown now
  • Support --json and --markdown in socket scan metadata
  • Support --json and --markdown in socket scan list

@pvdz pvdz requested a review from jdalton March 4, 2025 16:14
@jdalton jdalton merged commit 3e20170 into main Mar 4, 2025
15 checks passed
@jdalton jdalton deleted the scan_v2 branch March 4, 2025 17:10
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.

2 participants