yu210148 / gvm_install

A script to install GVM 20 or 21 on Ubuntu 20.04 or Debian 10.
GNU General Public License v3.0
142 stars 71 forks source link

missing "su gvm -c" at start of vt update command #84

Closed itiligent closed 2 years ago

itiligent commented 2 years ago

Hi, I just noticed on line 626 there seems to be a little error on one of the database update commands:

/opt/gvm/sbin/openvas --update-vt-info

I assume it should be:

su gvm -c "/opt/gvm/sbin/openvas --update-vt-info"

yu210148 commented 2 years ago

Thanks, I'll take a look when I can.

Kev.

On Wed, Jul 6, 2022, 8:38 AM itiligent @.***> wrote:

Hi, I just noticed on line 626 there seems to be a little error on one of the database update commands:

/opt/gvm/sbin/openvas --update-vt-info

I assume it should be:

su gvm -c "/opt/gvm/sbin/openvas --update-vt-info"

— Reply to this email directly, view it on GitHub https://github.com/yu210148/gvm_install/issues/84, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKFY4LQTCNJLM5BRTOGJGLVSV46FANCNFSM52ZT5WWQ . You are receiving this because you are subscribed to this thread.Message ID: @.***>

itiligent commented 2 years ago

also , on a related note in the same section, I noticed in the GVM docs that the feed update must be done in a specific order. see https://greenbone.github.io/docs/latest/21.04/source-build/index.html#performing-an-initial-feed-synchronization Adjustgin the script to update in the same order might help initial updates go more smoothly and quickly. (seems to go well in my testing) the order to allow for dependencies in the docs is:
vt (assume vt info also) SCAP CERT GVMD_DATA

yu210148 commented 2 years ago

Cool, kiddo is being a handful today. If you have a chance to put the changes into a pull request I could merge it from my phone otherwise it'll have to wait until I get time in front of my screen to do it.

Kev.

On Wed, Jul 6, 2022, 3:40 PM itiligent @.***> wrote:

also , on a related note in the same section, I noticed in the GVM docs that the feed update must be done in a specific order. see https://greenbone.github.io/docs/latest/21.04/source-build/index.html#performing-an-initial-feed-synchronization Adjustgin the script to update in the same order might help initial updates go more smoothly and quickly. (seems to go well in my testing) the order to allow for dependencies in the docs is: vt (assume vt info also) SCAP CERT GVMD_DATA

— Reply to this email directly, view it on GitHub https://github.com/yu210148/gvm_install/issues/84#issuecomment-1176619136, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKFY4LGCZ7DS5G5DFHHRSDVSXOKLANCNFSM52ZT5WWQ . You are receiving this because you commented.Message ID: @.***>