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

Create guidance on triaging build time dependency vulnerabilities #855

Closed
PushkarJ opened this issue Feb 2, 2022 · 9 comments
Closed

Create guidance on triaging build time dependency vulnerabilities #855

PushkarJ opened this issue Feb 2, 2022 · 9 comments
Labels
inactive No activity on issue/PR suggestion New suggestion for the CNCF sig-security group that don't fall into an existing category

Comments

@PushkarJ
Copy link
Collaborator

PushkarJ commented Feb 2, 2022

Description: Vulnerability scanners detect CVEs in build time dependencies. But, Best practices to triage these vulnerabilities are unclear

Impact: Adding docs based on experiences and anecdotes, that many projects can follow would be useful

Scope: Write best practices with examples like:

  • Focus on highest severity CVEs first
  • Document if not exploitable as a security advisory or github issue
  • Patch when exploitable and fixed by upstream dependency
  • Define roles and responsibilities

Meeting minutes where this was discussed: https://docs.google.com/document/d/170y5biX9k95hYRwprITprG6Mc9xD5glVn-4mB2Jmi2g/edit#heading=h.bssmkroi6sff and youtube recording: https://www.youtube.com/watch?v=MBHdvYW6YjI

cc @anvega @lumjjb @fkautz @ionut-arm

@PushkarJ PushkarJ added suggestion New suggestion for the CNCF sig-security group that don't fall into an existing category triage-required Requires triage labels Feb 2, 2022
@fkautz
Copy link
Contributor

fkautz commented Feb 2, 2022

Also make sure the time of the attestation is present. New information may change outcomes of the process.

@ionut-arm
Copy link
Contributor

I'll try to write-up my notes and thoughts on this, I'm currently digging a bit deeper in the Rust ecosystem for this.

@lumjjb
Copy link
Collaborator

lumjjb commented Feb 7, 2022

This could become a document to put under https://github.com/cncf/tag-security/tree/main/project-resources!

@fkautz
Copy link
Contributor

fkautz commented Feb 7, 2022 via email

@lumjjb lumjjb removed the triage-required Requires triage label Feb 16, 2022
@ionut-arm
Copy link
Contributor

Hi all, apologies for the long delay. I've made a markdown version with some guidance on build-time dependency vulnerabilities available here. Feel free to comment/modify as you please. It's fairly short, I wasn't sure just how much to go into detail and where.

@lumjjb
Copy link
Collaborator

lumjjb commented Feb 21, 2022

This is awesome @ionut-arm! Would you like to create a PR for this to the repo? This is definitely something the community at large can benefit from! Would be easier to get comments too!

@ionut-arm
Copy link
Contributor

Alright, I have finally opened that PR, many apologies for the delay

@stale
Copy link

stale bot commented Jun 18, 2022

This issue has been automatically marked as inactive because it has not had recent activity.

@stale stale bot added the inactive No activity on issue/PR label Jun 18, 2022
@anvega
Copy link
Collaborator

anvega commented Jun 21, 2023

#887 merged

@anvega anvega closed this as completed Jun 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
inactive No activity on issue/PR suggestion New suggestion for the CNCF sig-security group that don't fall into an existing category
Projects
None yet
Development

No branches or pull requests

5 participants