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

CHEF-6440: Integrates audit logging feature #6809

Merged
merged 33 commits into from
Nov 9, 2023
Merged

Conversation

Vasu1105
Copy link
Contributor

@Vasu1105 Vasu1105 commented Oct 19, 2023

Description

This PR introduces change for audit log configuration

Controls the feature through an environment variable. Set CHEF_PREVIEW_AUDIT_LOGGING to any non-empty value to enable audit logging. Optionally, you may also set the CLI option:

  • Adds --audit-log-location as cli option.
  • Sets the default log location to ~/.inspec/logs/inspec-audit-<timstamp>-<pid>.log for e.g. "inspec-audit-20231107T210740-67544.log"
  • Sends the required options to train for audit log generation

Related Issue

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New content (non-breaking change)
  • Breaking change (a content change which would break existing functionality or processes)

Checklist:

  • I have read the CONTRIBUTING document.

@Vasu1105 Vasu1105 requested a review from a team as a code owner October 19, 2023 05:54
@netlify
Copy link

netlify bot commented Oct 19, 2023

Deploy Preview for chef-inspec canceled.

Name Link
🔨 Latest commit b6c73fd
🔍 Latest deploy log https://app.netlify.com/sites/chef-inspec/deploys/654ceb423c92b80008ae60aa

@Vasu1105 Vasu1105 force-pushed the vj/integrate-audit-log branch 3 times, most recently from ed51ef7 to fc5df81 Compare October 26, 2023 10:21
@Vasu1105 Vasu1105 requested a review from a team as a code owner November 2, 2023 07:43
@github-actions github-actions bot added the Documentation ZH multi-repo label for the docs-team label Nov 2, 2023
@Vasu1105 Vasu1105 force-pushed the vj/integrate-audit-log branch 3 times, most recently from 5824eb8 to 50a42f0 Compare November 2, 2023 11:44
@Vasu1105 Vasu1105 changed the title [W.I.P] Adds cli options for audit log configuration Adds cli options for audit log configuration Nov 6, 2023
@Vasu1105 Vasu1105 changed the title Adds cli options for audit log configuration Integrates audit logging feature Nov 6, 2023
@Vasu1105 Vasu1105 changed the title Integrates audit logging feature CHEF-6440: Integrates audit logging feature Nov 6, 2023
@Vasu1105 Vasu1105 force-pushed the vj/integrate-audit-log branch 2 times, most recently from a6adc1b to 884035c Compare November 9, 2023 13:34
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
… only valid for inspec exec and inspec shell command as those commands use the backend to execute commands and for file operations

Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
…nable-audit-log option is provided in test

Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
…itly --enable-audit-log option is provided in test"

This reverts commit 50a42f0.
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
Vasu1105 and others added 25 commits November 9, 2023 19:48
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
…andle the enabling and disabling of audit log feature. And introduces few code changes after reverting renaming of --enable-audit-log option

Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
…on of audit log against the train changes made for audit log

Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com>
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
…ation/execution

Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
…so that unique audit log file created per invocation

Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
…ementation of audit log against the train changes made for audit log"

This reverts commit a66137e.
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
@clintoncwolfe clintoncwolfe merged commit 20d0a9d into main Nov 9, 2023
6 of 8 checks passed
@clintoncwolfe clintoncwolfe deleted the vj/integrate-audit-log branch November 9, 2023 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants