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

feat(RCSET-11195): Adding sdlc_snyk_container_scan #61

Conversation

SA-SecurityAutomation-OS
Copy link

@SA-SecurityAutomation-OS SA-SecurityAutomation-OS commented Apr 17, 2024

Context

The security team at OutSystems (Host & Network Security) has implemented image scanning at the repository level to enhance the security of the image build process.

  • Communications regarding this have been constantly provided and certain teams have added this image scanning as a workflow to their repository
  • We have identitifed this repository to be out-of-compliance by not implementing this workflow
  • So we are creating this branch by adding the necessary workflow configuration to integrate image scanning seamlessly into the development process.

What to do next:

  1. DONOT Merge this PR
  2. Git pull this branch
  3. Modify the workflow file name FROM security_sdlc_snyk_container_analysis.yaml TO sdlc_snyk_container_analysis.yaml so that you can control this file
  4. Commit the changes
  5. Push the changes
  6. Create a PR
  7. Merge the PR
  8. Delete the branch
  • If your PR is blocked by a CRITICAL/HIGH vulnerability that is fixable, you can ignore it for a fixed period of time by following this process

Jira Issue

RCSET-11195

Impacts

  1. Refactor (big refactor on a sensitive asset )
  2. Requires revision on public documentation
  3. Breaking Change
  • Behavioral breaking change
  • Breaks existent APIs
  1. Workflow will block all your PRs if snyk identifies a CRITICAL/HIGH vulnerability that can be fixed.

Code Checklist

  • Tested
  • Documented

Note:

  • If merging is blocked by The base branch requires all commits to be signed
  • please use the option Merge without waiting for requirements to be met (bypass branch protections) to bypass.

How to contact us

@SA-SecurityAutomation-OS SA-SecurityAutomation-OS requested a review from a team as a code owner April 17, 2024 20:52
@dnlopes
Copy link
Contributor

dnlopes commented Apr 18, 2024

This repository does not push images to Eden. I will add the exemption topic to be added here.

@dnlopes dnlopes closed this Apr 18, 2024
@Matrikss Matrikss deleted the feature/RCSET-11195/security_snyk_enforcement branch August 26, 2024 11:09
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.

2 participants