Skip to content
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.

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

7zip Easy Unzip Improvements #1018

Closed
nadrojisk opened this issue Apr 26, 2024 · 3 comments · Fixed by #1025
Closed

7zip Easy Unzip Improvements #1018

nadrojisk opened this issue Apr 26, 2024 · 3 comments · Fixed by #1025
Assignees
Labels
💎 enhancement It is working, but it could be better ❔ discussion Further discussion is needed

Comments

@nadrojisk
Copy link

Details

Details

I suggest:

  • At the moment there is a handy unzip with "infected" right click context menu that utilizes 7zip. I suggest changing it to the following:
 VM-Add-To-Right-Click-Menu $toolName 'unzip "infected"' "`"$7zExecutablePath`" x -pinfected `"%1`"" "$executablePath" -extension $extension
  • I suggest swapping e to x as it extract with full paths. Sometimes there are nested directories within a ZIP and it can be cumbersome if those are just flattened upon extracting.
  • I think it would also be great to extract the archive into a directory with its same name similar to how when you right click an archive it says extract to [Archive Name]\
@Ana06
Copy link
Member

Ana06 commented Apr 29, 2024

@nadrojisk thanks your sharing your improvement ideas!

I suggest swapping e to x as it extract with full paths. Sometimes there are nested directories within a ZIP and it can be cumbersome if those are just flattened upon extracting.

I agree with this one!

I think it would also be great to extract the archive into a directory with its same name similar to how when you right click an archive it says extract to [Archive Name]\

  • How do you propose to implement this? The -o option allows to provide a directory name, but I am not sure how to get the zip name in the registry command. At the moment we use the argument %1.

  • Most of the zips with infected password I use are just a single malware binary zipped or a folder inside a zip. In both cases, I would personally find the extra directory annoying. @mandiant/flare-vm what do others think?

@Ana06 Ana06 added 💎 enhancement It is working, but it could be better ❔ discussion Further discussion is needed 😕 needs info Further information is needed 🌀 FLARE-VM A package or feature to be used by FLARE-VM and removed 🌀 FLARE-VM A package or feature to be used by FLARE-VM labels Apr 29, 2024
@mr-tz
Copy link
Contributor

mr-tz commented Apr 29, 2024

Agreed on using x. No hard feelings on adding an outer directory.

@nadrojisk
Copy link
Author

Agreed on using x. No hard feelings on adding an outer directory.

No hard feelings at all :) Thanks for adding the x change though, you all are great!

@Ana06 Ana06 self-assigned this Apr 30, 2024
@Ana06 Ana06 removed the 😕 needs info Further information is needed label Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💎 enhancement It is working, but it could be better ❔ discussion Further discussion is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants