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

complete consolidation of Safety ignore-lists #14

Open
cfm opened this issue Nov 6, 2023 · 3 comments
Open

complete consolidation of Safety ignore-lists #14

cfm opened this issue Nov 6, 2023 · 3 comments

Comments

@cfm
Copy link
Member

cfm commented Nov 6, 2023

Witness what's been needed to ignore Safety 61601 (urllib3 CVE-2023-43804) to date:

@cfm cfm mentioned this issue Nov 6, 2023
5 tasks
@legoktm
Copy link
Member

legoktm commented Nov 6, 2023

Now that we have fpf-misc-resources is there value in the having per-repository safety jobs? My initial impression is that there isn't - the only thing it would catch is if we were adding a new library that happened to have a security vuln, but there's no guarantee it would get caught because CI uses the free safety repo that is up to a month behind, while the fpf-misc-resources job uses the up-to-date version.

@zenmonkeykstop
Copy link

I do think we should pick one - but I'm not sure this is the one that I would pick. In much the same way as we're looking at moving debian/s back into their respective repos to reduce overhead, these ignore lists relate to dependency manage within individual projects and are probably going to be easier to manage there. But for sure, the ~month delay is a pain.

@legoktm
Copy link
Member

legoktm commented Jan 3, 2025

We could also keep the per-repo safety jobs but use the ignore list from the fpf-misc-resources repository (just checkout the repo during the test run and make sure safety uses it for the exclude list instead of all of our --ignore lines.


In the meantime I've been using this script for a while now:

#!/usr/bin/env python3

import json
from pathlib import Path

data = json.loads(Path("projectfiles/securedrop.json").read_text())
for id in sorted(data["variables"]["SAFETY_IGNORE_IDS"]):
    print(f"		--ignore {id} \\")

to generate the Makefile listing.

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

No branches or pull requests

3 participants