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

Merge VM-Install-From-Zip and VM-Install-Raw-GitHub-Repo #749

Closed
Ana06 opened this issue Nov 22, 2023 · 7 comments · Fixed by #990
Closed

Merge VM-Install-From-Zip and VM-Install-Raw-GitHub-Repo #749

Ana06 opened this issue Nov 22, 2023 · 7 comments · Fixed by #990
Assignees
Labels
💎 enhancement It is working, but it could be better ❔ discussion Further discussion is needed

Comments

@Ana06
Copy link
Member

Ana06 commented Nov 22, 2023

Details

From https://github.com/mandiant/VM-Packages/pull/727>

The code of the VM-Install-From-Zip and VM-Install-Raw-GitHub-Repo is very similar, which is not surprising as a GitHub-Raw install is the same as installing a Zip.

Any thoughts about merging these two functions?

@Ana06 Ana06 added 💎 enhancement It is working, but it could be better ❔ discussion Further discussion is needed labels Nov 22, 2023
@Ana06 Ana06 self-assigned this Nov 22, 2023
@Ana06 Ana06 added this to the FLARE-VM 2023 Q4 milestone Nov 22, 2023
@Ana06
Copy link
Member Author

Ana06 commented Dec 12, 2023

It may make sense to address it together with #674

@day1player
Copy link
Contributor

What about configuring an alias? Then we can leave all packages as they are. I like that they are named separately since people might not know automatically that they are the same

@Ana06
Copy link
Member Author

Ana06 commented Mar 4, 2024

A raw GH package is just a Zip that we download from GH. There is no difference at all with Zips we download from somewhere else. I think having both causes confusion. VM-Install-From-Zip has been lately improved and supports more options. I would like to get rid of VM-Install-Raw-GitHub-Repo both in the code and in the issues template and script to create packages. I think this will allow us to add some of the options to the issue template easily.

Proposal to progressively implement this (in different PR to be sent only after the previous steps have been merged):

  • Merge VM-Install-From-Zip and VM-Install-Raw-GitHub-Repo into VM-Install-From-Zip but leave VM-Install-Raw-GitHub-Repo as a wrapper/alias that just calls VM-Install-From-Zip so that we don't have to modify all the packages using VM-Install-Raw-GitHub-Repo
  • Modify the script to generate packages, the issue template, and GH integration to only use VM-Install-From-Zip
  • Update all the packages to use VM-Install-From-Zip

@Ana06
Copy link
Member Author

Ana06 commented Mar 4, 2024

@emtuls would you like to work on this? You have worked on the shortcuts lately that are related. 😉

@Ana06
Copy link
Member Author

Ana06 commented Apr 12, 2024

This issue is not fixed. WE have simplified VM-Install-Raw-GitHub-Repo in #959, but the objective is to get rid of it completely. Steps needed to close this PR:

  • Modify the script to generate packages, the issue template, and GH integration to only use VM-Install-From-Zip
  • Update all the packages to use VM-Install-From-Zip

@Ana06
Copy link
Member Author

Ana06 commented Apr 12, 2024

@Ana06
Copy link
Member Author

Ana06 commented Apr 12, 2024

minidump fails to install, so documented the existent issue to fix it, as it is unrelated with this issue: #830 (comment)

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