search

joanbono / Snorter

Snort + Barnyard2 + Pulledpork → The easy way!
https://joanbono.github.io/Snorter
GNU General Public License v3.0
166 stars 34 forks source link

Problem with sudo in Debian #18

Closed andreumont closed 6 years ago

andreumont commented 7 years ago

Hello!

When you try to install snorter packages on Debian systems, there appear some errors because sudo isn't installed in the OS by default.

Maybe you can change the script to let people install snorter without the requirement to have sudo installed.

Many thanks for your apport to comunity. Good job bro!

joanbono commented 7 years ago

I'll check this, but I think is preferrable to install sudo and add the user to the sudoers, because all the other Debian-based distros come with sudo installed by default.

Anyway, feel free to open a Pull Request if you want to contribute!

I'll check this.

Good catch!

Thanks!! 😄

HarriL commented 7 years ago

Installation of sudo should be in the Debian distro, not in a software installation. It is an security issue over all.

Perhaps you should not use sudo inside the script, but outside?

  1. inside in a script user/installed does not know that there is root priviledge used because it is hidden
  2. during a long execution the sudo it may ask password and may silently fail
joanbono commented 7 years ago

Thanks @HarriL, yes, you're rigth. I'll keep an eye on this when I finish my holidays.

Please stay tuned, I may need some advice on this.

Thanks a lot :D