Skip to content

Stop special-casing RUSTSEC-2025-0021 for cargo deny on CI #1991

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

Merged
merged 1 commit into from
May 5, 2025

Conversation

EliahKagan
Copy link
Member

@EliahKagan EliahKagan commented May 5, 2025

This reverts commit 67d9bf4, but not the other changes from #1927.

gix-testtools 0.16.1 has the change in 9b12d50 (#1972) from depending on previous SemVer-incompatible versions of gix-* crates to depending on the current versions. Since then, nothing affected by RUSTSEC-2024-0436 appears in our dependency tree, and it is no longer necessary or desirable to use a more complicated cargo deny check advisories scanning approach allowing that advisory through gix-testtools.


The scan looks good. As expected, even without special casing that advisory, the advisories check passes.

Note that this is not related to #1989, except in that they are both unblocked by #1972.

…tools"

This reverts commit 67d9bf4, but
not the other changes from GitoxideLabs#1927.

`gix-testtools` 0.16.1 has the change in 9b12d50 (GitoxideLabs#1972) from
depending on previous SemVer-incompatible versions of `gix-*`
crates to depending on the current versions. Since then, nothing
affected by https://rustsec.org/advisories/RUSTSEC-2024-0436.html
appears in our dependency tree, and it is no longer necessary or
desirable to use a more complicated `cargo deny check advisories`
scanning approach allowing that advisory through `gix-testtools`.
@EliahKagan EliahKagan changed the title Revert "On CI, have cargo deny allow RUSTSEC-2025-0021 via gix-testtools" Stop special-casing RUSTSEC-2025-0021 for cargo deny on CI May 5, 2025
@EliahKagan EliahKagan marked this pull request as ready for review May 5, 2025 01:07
@EliahKagan EliahKagan enabled auto-merge May 5, 2025 01:07
@EliahKagan EliahKagan merged commit bcdcba3 into GitoxideLabs:main May 5, 2025
22 checks passed
@EliahKagan EliahKagan deleted the deny-revert branch May 5, 2025 01:21
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.

1 participant