Skip to content

Commit b1d82a6

Browse files
committed
Replace pipenv check with pip-audit
Why these changes are being introduced: As of pipenv 2025.0.1 the use of `pipenv check` would throw an error, indicating that the library `safety` was not installed. It worked to run `pipenv check --auto-install` which would temporarily install `safety`, but this was not ideal for multiple reasons. First, we anticipate potentially moving away from `pipenv`. Second, it appears that `safety` is moving to a pay / subscription model. Third, it remains a little obfuscated what `pipenv check` is actually doing. As this new situation affects all builds in Github Actions CI, we need a way to scan for vulnerabilities that ideally is not a massive overhaul of our vulnerability scanning approach. How this addresses that need: `pip-audit` is a nice standalone, open-source library that performs very similar work to `safety`. This commit replaces `pipenv check` (which was `safety` under the hood) with `pip-audit`. Side effects of this change: * Builds will be successful in Github Actions Relevant ticket(s): * https://mitlibraries.atlassian.net/browse/IN-1240
1 parent c2e3f1e commit b1d82a6

File tree

5 files changed

+1179
-926
lines changed

5 files changed

+1179
-926
lines changed

.pre-commit-config.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,8 @@ repos:
2222
language: system
2323
pass_filenames: true
2424
types: ["python"]
25-
- id: safety
26-
name: safety
27-
entry: pipenv check --ignore 70612
25+
- id: pip-audit
26+
name: pip-audit
27+
entry: pipenv run pip-audit
2828
language: system
2929
pass_filenames: false

Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ ruff: # Run 'ruff' linter and print a preview of errors
5959
pipenv run ruff check .
6060

6161
safety: # Check for security vulnerabilities and verify Pipfile.lock is up-to-date
62-
pipenv check --ignore 70612
62+
pipenv run pip-audit
6363
pipenv verify
6464

6565
lint-apply: # Apply changes with 'black' and resolve 'fixable errors' with 'ruff'

Pipfile

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ pre-commit = "*"
2424
pytest = "*"
2525
ruff = "*"
2626
types-requests = "*"
27+
pip-audit = "*"
2728

2829
[requires]
2930
python_version = "3.12"

0 commit comments

Comments
 (0)