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

Add parameter to suppress the '--config /etc/aide.conf' argument in the cron job. #14

Merged
merged 2 commits into from
Aug 5, 2022
Merged

Add parameter to suppress the '--config /etc/aide.conf' argument in the cron job. #14

merged 2 commits into from
Aug 5, 2022

Conversation

bschonec
Copy link
Contributor

@bschonec bschonec commented Jul 14, 2022

I'm working with 3rd party vendors doing CIS and security checking. The current CRON job:

0 0 * * * root nice ionice -c3 /usr/sbin/aide --config /etc/aide.conf --check

fails the regex tests that the vendors do for AIDE due to '--config /etc/aide.conf' being specified. I figure it's easier to update this module than to convince the vendors to update their regular expressions.

The regex used by CIS/STIG is:

pattern: ^\s*([^#]+\s+)?\/usr\/sbin\/aide\s(--?\S+\s)*--(check|update)\b.*$

I propose an option to exclude the '--config /etc/aide' part of the CRON job and just let aide use its default of /etc/aide.conf.

@kekogya kekogya added the question Further information is requested label Jul 29, 2022
@kekogya
Copy link
Contributor

kekogya commented Jul 29, 2022

@bschonec, sorry for the long wait. I had a high priority task come up that needs my attention so it might take me a little while before I switch gears to take a look at this pull request. Once I start taking a look, I will let you know if I have any questions.

@kekogya kekogya added enhancement New feature or request and removed question Further information is requested labels Aug 4, 2022
@kekogya
Copy link
Contributor

kekogya commented Aug 5, 2022

@bschonec thank you for your contribution to this module and sorry for the long wait. I will work on a new release soon.

@kekogya kekogya merged commit 37d643e into indiana-university:master Aug 5, 2022
@kekogya kekogya self-assigned this Aug 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants