Allow paste
even though unmaintained, for now
#1879
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
https://rustsec.org/advisories/RUSTSEC-2024-0436.html now causes the
cargo deny advisories
check to fail (even if the separate and more important failure fromring
is fixed by bumping thering
version, as in #1878).paste
is mature and would be hard to remove as a transitive dependency at this time:As discussed in rustsec/advisory-db#2215 and leptos-rs/leptos#3685,
paste
is widely used and there is community interest in maintaining it.When the status changes or more information about the future of
paste
or its alternatives is available,deny.toml
could be updated again (even if only with a comment).This PR adds a commit atop #1878. It would be reasonable to include this change there, but I cannot use a review comment to propose an automatically appliable patch to code in a PR that is not changed or right next to lines that are changed. I considered opening this against the Dependabot branch for #1878 rather than against main, but in this case it seems like that might be more complicated to handle; but I'd be pleased to change the base branch on request.
If #1878 is merged first, then this can be merged and the history should be okay and still free of duplicate commits. Or this could be rebased after that for a slightly clearer history. Or if this is merged before #1878, it will bring in the changes from there, and I believe #1878 will be closed automatically. Another option is to merge this commit into the branch for #1878 (
git merge 1d9f7cd
when on that branch) and then merge #1878.This causes the
cargo deny advisories
check to pass. The remaining failures here are unrelated to the changes. They are the same as the other failures occurring on main:test-fixtures-windows
(due to #1849, #1870 would fix) andtest-32bit
(due to rustup changes, #1874 would fix).