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(audit-logs): Redact secret data #508

Merged
merged 3 commits into from
Aug 30, 2024
Merged

fix(audit-logs): Redact secret data #508

merged 3 commits into from
Aug 30, 2024

Conversation

uwe-mayer
Copy link
Contributor

Description

This redacts secret data in admission request logging

  • πŸ• Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation Update
  • 🎨 Style
  • πŸ§‘β€πŸ’» Code Refactor
  • πŸ”₯ Performance Improvements
  • βœ… Test
  • πŸ€– Build
  • πŸ” CI
  • πŸ“¦ Chore (Release)
  • ⏩ Revert

Related Tickets & Documents

Added tests?

No direct way to test controller output logs with Ginkgo. Ginkgo swallows logs when testing.

@uwe-mayer uwe-mayer requested a review from a team as a code owner August 30, 2024 06:59
Copy link
Contributor

@abhijith-darshan abhijith-darshan left a comment

Choose a reason for hiding this comment

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

πŸš€

@abhijith-darshan
Copy link
Contributor

@uwe-mayer - for ginkgo logging we can pass these parameters to go test - -ginkgo.v -ginkgo.progress -test.v

@uwe-mayer
Copy link
Contributor Author

As discussed in Slack we will completely remove Object and OldObject from the logs to reduce noise.
Relevant information is already on the request

@uwe-mayer uwe-mayer merged commit c123ad3 into main Aug 30, 2024
5 checks passed
@uwe-mayer uwe-mayer deleted the fix/reduce-audit-logs branch August 30, 2024 11:24
andypf pushed a commit that referenced this pull request Sep 4, 2024
* fix(audit-logs): Redact secret data

* Fix linting

* Remove Object and OldObject completely from log
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.

3 participants