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

Set hostname as computer node name in audit logs. #4701

Merged
merged 1 commit into from
Aug 13, 2019

Conversation

adelton
Copy link
Collaborator

@adelton adelton commented Aug 9, 2019

Description:

  • Set hostname as computer node name in audit logs.

Rationale:

  • If option name_format is left at its default value of none, audit events from different computers may be hard to distinguish.

@adelton adelton changed the title Set number of records to cause an explicit flush to audit logs. Set hostname as computer node name in audit logs. Aug 12, 2019
@jan-cerny jan-cerny self-assigned this Aug 12, 2019
@jan-cerny
Copy link
Collaborator

@adelton The patch looks good to me, but please resolve the conflicting files.

@adelton
Copy link
Collaborator Author

adelton commented Aug 12, 2019

Please start with #4695 -- that one has already been rebased, and since the other pull requests conflict with each other, we really need to do it in the original order. I'll rebase once #4695 is in.

@jan-cerny
Copy link
Collaborator

@adelton OK, I'll review #4695 first.

@adelton
Copy link
Collaborator Author

adelton commented Aug 13, 2019

Thanks. Rebased on master -> 59c9e7e.

@jan-cerny jan-cerny added this to the 0.1.46 milestone Aug 13, 2019
@jan-cerny jan-cerny merged commit 87ede86 into ComplianceAsCode:master Aug 13, 2019
This pull request was closed.
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