Skip to content

Conversation

@jozef-sabo
Copy link
Contributor

Closes #25

@jozef-sabo jozef-sabo marked this pull request as draft April 14, 2025 08:30
@jozef-sabo jozef-sabo changed the title Initial commit Skip AVs detection Apr 14, 2025
This allows to skip the initial detection of all the installed Antivirus solutions and go directly to files extraction
@jozef-sabo jozef-sabo marked this pull request as ready for review April 22, 2025 08:06
Copy link
Collaborator

@vojtechjelinek vojtechjelinek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! One comment.

Comment on lines 159 to 164
parser.add_argument(
"-r",
"--all-avs",
action="store_true",
help="skip detection of existing avs and try every supported one",
)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would check all AVs by default. So probably add an option to only analyze the installed AVs.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done. The behaviour will make sence only after #22 is applied.

Copy link
Collaborator

@vojtechjelinek vojtechjelinek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Collaborator

@vojtechjelinek vojtechjelinek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@vojtechjelinek vojtechjelinek merged commit 464c62b into NUKIB:main Apr 24, 2025
5 checks passed
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.

Skip AV detection and rather try all AVs

2 participants