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.

Sign up for GitHub

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

Jump to bottom

Set number of records to cause an explicit flush to audit logs. #4697

Merged
merged 1 commit into from
Aug 14, 2019
Merged

Set number of records to cause an explicit flush to audit logs. #4697

merged 1 commit into from
Aug 14, 2019

Conversation

adelton
Copy link
Collaborator

@adelton adelton commented Aug 9, 2019

Description:

  • Set number of records to cause an explicit flush to audit logs.

Rationale:

  • If option freq isn't set to 50, the flush to disk may happen after higher number of record, increasing the danger of audit loss.

@adelton
Copy link
Collaborator Author

adelton commented Aug 9, 2019

Fixed typo -> 5ac8f19.

jan-cerny
jan-cerny reviewed Aug 9, 2019
View reviewed changes
linux_os/guide/system/auditing/auditd_freq/rule.yml Outdated

rationale: |-
If option <tt>freq</tt> isn't set to <tt>50</tt>, the flush to disk
may happen after higher number of record, increasing the danger
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

probably records

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in 41a2deb.

@adelton
Copy link
Collaborator Author

adelton commented Aug 13, 2019

Rebased on master -> 111ee16.

@jan-cerny jan-cerny self-assigned this Aug 13, 2019
@jan-cerny jan-cerny added this to the 0.1.46 milestone Aug 13, 2019
@jan-cerny jan-cerny merged commit 78a9994 into ComplianceAsCode:master Aug 14, 2019
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jan-cerny jan-cerny jan-cerny left review comments

Assignees

@jan-cerny jan-cerny

Labels
None yet
Projects
None yet
Milestone
0.1.46
Development

Successfully merging this pull request may close these issues.
2 participants
@adelton @jan-cerny