Installer cve search database #67
Conversation
|
Thanks for fixing our installer. I will check this tomorrow |
| @@ -299,7 +299,7 @@ case ${ANSWER:0:1} in | |||
| docker pull fkiecad/fact_extractor:latest | |||
| fi | |||
| if ! [[ -f "./external/extract.py" ]]; then | |||
| download_file "FACT-extract" "https://raw.githubusercontent.com/fkie-cad/fact_extractor/master/extract.py" "external/extract.py" | |||
There was a problem hiding this comment.
I am wondering why the download_file should not work in this situation. I will try another fresh installation to check on this
| @@ -369,6 +369,8 @@ print_file_info "$BINUTIL_VERSION_NAME" "The GNU Binutils are a collection of bi | |||
| print_tool_info "texinfo" 1 | |||
| print_tool_info "gcc" 1 | |||
| print_tool_info "build-essential" 1 | |||
| print_tool_info "gawk" 1 | |||
There was a problem hiding this comment.
We should start commenting from which other tool/module this is needed. Could you please add this here.
We are going to start with the other tools
| @@ -493,10 +524,10 @@ case ${ANSWER:0:1} in | |||
| y|Y ) | |||
| sudo cve_searchsploit -u | |||
There was a problem hiding this comment.
do we really need this sudo here?
Should we run such an update also outside of the docker environment. Otherwise the first run will do it and it will take a while during the first emba run.
| echo -e "\\n""$MAGENTA""$BOLD""For using CVE-search you have to install all the requirements and the needed database.""$NC" | ||
| echo -e "$MAGENTA""$BOLD""Installation instructions can be found on github.io: https://cve-search.github.io/cve-search/getting_started/installation.html#installation""$NC" | ||
| # echo -e "\\n""$MAGENTA""$BOLD""For using CVE-search you have to install all the requirements and the needed database.""$NC" | ||
| # echo -e "$MAGENTA""$BOLD""Installation instructions can be found on github.io: https://cve-search.github.io/cve-search/getting_started/installation.html#installation""$NC" |
There was a problem hiding this comment.
Looks good to me. Please remove these lines completely.
|
Hi @Anemosx, great work. Thank you. The installer is now able to bring up a fully working emba installation including cve-search with the needed database. A first run after installation gives the expected results:
Could you please take a quick look at the comments and afterwards we are ready to merge. |
|
@p4cx I will merge this now. Could you please include the comments in your cleanup? Thanks |
Emba installer downloads cve serach database:
Download of cve database within the installer Installer does not install CVE-database #60
Tested on Kali Linux 2021.1 Kali Linux 2021.1 tests #61