Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: add detection for codebuild public projects #302

Merged
merged 6 commits into from
Sep 2, 2021
Merged

feat: add detection for codebuild public projects #302

merged 6 commits into from
Sep 2, 2021

Conversation

rileydakota
Copy link
Contributor

Background

AWS recently added the capability of creating public codebuild projects - which can expose source code, build artifacts, environment variables, and secrets accidentally. This PR adds a detection for the CodeBuild UpdateProjectVisibility API

Changes

  • Adds a new rule for UpdateProjectVisibility API call that sets a project to PUBLIC_READ

Testing

  • 3 Test Cases added

Copy link
Contributor

@nhakmiller nhakmiller left a comment

Choose a reason for hiding this comment

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

Looks good to me!

aws_cloudtrail_rules/aws_codebuild_made_public.yml Outdated Show resolved Hide resolved
Copy link
Contributor

@lindsey-w lindsey-w left a comment

Choose a reason for hiding this comment

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

A very useful rule!

aws_cloudtrail_rules/aws_codebuild_made_public.py Outdated Show resolved Hide resolved
rileydakota and others added 4 commits August 19, 2021 09:39
Co-authored-by: Lindsey Whitehurst <43453975+lindsey-w@users.noreply.github.com>
@nhakmiller nhakmiller requested a review from bseb as a code owner September 2, 2021 23:45
@nhakmiller nhakmiller merged commit 0b926db into panther-labs:master Sep 2, 2021
melenevskyi pushed a commit that referenced this pull request Dec 12, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants