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

update(BoM): update BoM queries and BoM docs #5074

Merged
merged 3 commits into from
Mar 29, 2022

Conversation

rafaela-soares
Copy link
Contributor

Proposed Changes

  • split policy from resource_accessibility (It was decided that KICS does not have the responsibility to interpret the policy. KICS should only display it, when it exists)
  • unified some bom_output fields
  • updated BoM docs
  • commented user_name and is_default_policy (for now, these fields are not used for CE and maybe they will not be used in the future. Anyway, it considers only one user and it can be more than one)

I submit this contribution under the Apache-2.0 license.

@kicsbot
Copy link
Contributor

kicsbot commented Mar 28, 2022

Scan submitted to Checkmarx

@kicsbot
Copy link
Contributor

kicsbot commented Mar 28, 2022

Logo
Checkmarx SAST - Scan Summary & Details

Cx-SAST Summary

Total of 5 vulnerabilities
High 0 High
Medium 0 Medium
Low 5 Low
Info 0 Info

Violation Summary

No policy violation found

Copy link
Collaborator

@cx-joao-reigota cx-joao-reigota left a comment

Choose a reason for hiding this comment

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

LGTM

@cx-joao-reigota cx-joao-reigota merged commit ef26485 into master Mar 29, 2022
@cx-joao-reigota cx-joao-reigota deleted the update/terraform_aws_bom branch March 29, 2022 11:44
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.

3 participants