Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix remote script a #14

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

eyalkraft
Copy link

@eyalkraft eyalkraft commented Nov 18, 2022

Some changes in the used tools resulted with red-detector not working (failing on fetching the databases).

The problems & fixes are:

  • ftp://ftp.pangeia.com.br/pub/seg/pac/chkrootkit.tar.gz is down. Instead downloading from Github.
  • vuls tools changed the command format, for example goval-dictionary fetch-redhat changed to be goval-dictionary fetch redhat. I updated the commands to the new format.
  • vuls tools changed the docker image working directory names to match the tool name, for example in vuls/go-cve-dictionary docker image, /vuls changed to be /goval-dictionary. I updated the docker mounts to match the new names.

Additional fix to one of the used tools

@eyalkraft
Copy link
Author

😓 Just saw #12 with some of these fixes...
Worth merging one of them to have red-detector work out of the box.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant