Skip to content

Add VirusTotal scan action workflow#458

Draft
jjw24 wants to merge 14 commits into
mainfrom
add_virus_toal_scan
Draft

Add VirusTotal scan action workflow#458
jjw24 wants to merge 14 commits into
mainfrom
add_virus_toal_scan

Conversation

@jjw24

@jjw24 jjw24 commented Feb 4, 2025

Copy link
Copy Markdown
Member

@jjw24 jjw24 self-assigned this Feb 4, 2025
@jjw24 jjw24 marked this pull request as draft February 4, 2025 10:27
@cibere

cibere commented Feb 5, 2025

Copy link
Copy Markdown
Member

Well... this would fail when testing my pyrepl plugin 😓

@jjw24

jjw24 commented Feb 5, 2025

Copy link
Copy Markdown
Member Author

Hmm, not sure what you mean, this would not have any impact on testing. It won't be run as part of testing.

@cibere

cibere commented Feb 5, 2025

Copy link
Copy Markdown
Member

Hmm, not sure what you mean, this would not have any impact on testing. It won't be run as part of testing.

The intention is to run the plugin's extracted files through virustotal, right? If you do that with my pyrepl plugin, it will be flagged due to the standalone exe

@jjw24

jjw24 commented Feb 7, 2025

Copy link
Copy Markdown
Member Author

The intention is to run the plugin's extracted files through virustotal, right?

Yes.

If you do that with my pyrepl plugin, it will be flagged due to the standalone exe

Is it flagging because it's standalone exe or something else though?

@cibere

cibere commented Feb 7, 2025

Copy link
Copy Markdown
Member

If you do that with my pyrepl plugin, it will be flagged due to the standalone exe

Is it flagging because it's standalone exe or something else though?

It's because of the way pyinstaller makes the standalone exe. My understanding is that it uses methods similar to trojans, so virustotal flags it. Pyinstaller doesn't have to build it as a standalone exe though, that was just the choice I made when making the build workflow.

@jjw24 jjw24 changed the base branch from plugin_api_v2 to main September 4, 2025 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants