Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Default logdir is in incorrect #8

Open
nbrys opened this issue Feb 4, 2021 · 1 comment
Open

Default logdir is in incorrect #8

nbrys opened this issue Feb 4, 2021 · 1 comment

Comments

@nbrys
Copy link

nbrys commented Feb 4, 2021

Hi,
Currently the default for the log_dir points to:

Defaults to '/var/log/suricata

But, looking at the suricata codebase for the logrotate script: https://github.com/OISF/suricata/blob/master-4.1.x/etc/suricata.logrotate.in

This will translate to:

# Sample /etc/logrotate.d/suricata configuration file. /var/log/suricata*.log /var/log/suricata*.json { daily missingok rotate 5 compress delaycompress minsize 500k sharedscripts postrotate /bin/kill -HUP cat /var/run/suricata.pid 2> /dev/null 2> /dev/null || true endscript }

Which is incorrect. There is a / missing behind /var/log/suricata

@odinje
Copy link
Owner

odinje commented Feb 5, 2021

Hi,

Logroate this not handled by this module, and log_dir in this repo should are not using the variables in the Suricata repo. These variables are used by Suricata when building from source, but Suricata is by default installed from repositories within this module

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

No branches or pull requests

2 participants