Skip to content
This repository has been archived by the owner on Jun 9, 2022. It is now read-only.

Dynamically included modules #35

Open
scooterhanson opened this issue Nov 21, 2017 · 2 comments
Open

Dynamically included modules #35

scooterhanson opened this issue Nov 21, 2017 · 2 comments
Labels
enhancement Feature requests. help wanted The maintainers would appreciate help from the community in resolving these issues.

Comments

@scooterhanson
Copy link
Contributor

It would be awesome if modules could be included/loaded dynamically without having to fetch from github. Maybe like declaring dependencies in the rc file.

@Stono Stono added the enhancement Feature requests. label Dec 2, 2017
@felixhammerl
Copy link
Collaborator

I've thought about that and it would be awesome. Actually, apart from the hours to invest it would also be relatively trivial. Currently thinking about spinning out all of the existing modules into npm dependencies. Instead of specifying them as -m files-contents it would be something like -m @hawkeyesec/modules-files-contents, or something along those lines.

Should be even more straightforward after the massive refactoring, if you wanna give it a shot.

The only downside is the dependency to the scanner through the injected FileManager ... not sure how to solve that one.

@felixhammerl felixhammerl added the help wanted The maintainers would appreciate help from the community in resolving these issues. label Oct 19, 2018
@bekh6ex
Copy link
Contributor

bekh6ex commented Jul 11, 2019

The only downside is the dependency to the scanner through the injected FileManager ... not sure how to solve that one.

I think that can be solved by extracting it into a separate package like @hawkeyesec/core on which modules will depend.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Feature requests. help wanted The maintainers would appreciate help from the community in resolving these issues.
Projects
None yet
Development

No branches or pull requests

4 participants