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

Fix log file check #451

Merged
merged 2 commits into from
Mar 8, 2021
Merged

Conversation

ArnoStiefvater
Copy link
Member

What:

Adds missing handling of syslog logging and empty file string to #447 .

Why:

Else not able to init openvas with some logging configs.

How:

add the following to the openvas_log.conf

[sd   main]
prepend=%t %s %p
separator=:
prepend_time_format=%Y-%m-%d %Hh%M.%S %Z
file=syslog
syslog_facility=daemon
syslog_indent=openvas
level=127

Without the patch openvas -s will fail if openvas is started as other user (e.g. sudo -u testuser openvas -s). Same if file= is put into the config. With applied PR this is fixed.

Checklist:

Syslog as file and empyt str as file in the
openvas_log.conf is not to be handled as error.
@bjoernricks bjoernricks merged commit d60bfe3 into greenbone:master Mar 8, 2021
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

Successfully merging this pull request may close these issues.

2 participants