Skip to content
This repository has been archived by the owner on Apr 3, 2024. It is now read-only.

Update SecretManager.yaml #173

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hashcat3
Copy link
Contributor

@hashcat3 hashcat3 commented Jul 8, 2022

T1552.001 (Credentials In Files) should be mapped here. It is mapped against cloudkeymanagement.yaml already, but may be appropriate against this control as well.

T1552.001 (Credentials In Files) should be mapped here. It is mapped against cloudkeymanagement.yaml, but may be appropriate against this control as well.
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jul 8, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link

@drjohnson87 drjohnson87 left a comment

Choose a reason for hiding this comment

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

I agree that T1552.001 should be mapped to this technique. With Secret manager, Cloud Audit Logs integration interacts with one another and generates an audit log. This integration makes meeting audit and compliance requirements easy. Auditing can preemptively search for files containing passwords and take actions to reduce the exposure risk when found.

@mehaase
Copy link
Contributor

mehaase commented Jul 18, 2022

The category, value, and comments are empty. See the build log for details.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants