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

Consolidate sandbox images and inject analyze script at runtime #301

Open
calebbrown opened this issue Jun 23, 2022 · 1 comment
Open

Consolidate sandbox images and inject analyze script at runtime #301

calebbrown opened this issue Jun 23, 2022 · 1 comment
Labels
dynamic analysis Issues specific to the implementation of Dynamic Analysis enhancement New feature or request needs discussion

Comments

@calebbrown
Copy link
Contributor

Packages downloaded from package repositories often depend on other ecosystems to function correctly (e.g. an NPM package uses PyPI).

So rather than having a separate sandbox image for each ecosystem we could instead have a single common image and just mount the analyze script into the image.

This enables some interesting other capabilities:

  • make the sandbox harder to evade by detecting the analyze script
  • allows different versions of linux, node, python, etc to be tested
  • makes the infrastructure more flexible for broader use-cases
@calebbrown calebbrown added the enhancement New feature or request label Jun 23, 2022
@calebbrown
Copy link
Contributor Author

This needs some more thought.

One challenge is handling packages that depend on a specific version of Node, PHP, Python, etc.

Perhaps more decoupling between the sandboxes + dynamic analysis and the package repositories + analysis scripts makes sense though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dynamic analysis Issues specific to the implementation of Dynamic Analysis enhancement New feature or request needs discussion
Projects
None yet
Development

No branches or pull requests

2 participants