chore(deps): unblock dev CI — ignore RUSTSEC-2026-0247, patch russh/quinn-proto, resync Cargo.lock - #387
Merged
Conversation
…uinn-proto, resync Cargo.lock
Three coupled fixes that together restore a mergeable `dev`.
1. cargo-deny (required check) is failing on every open PR and on `dev`
itself with a single finding:
error[unmaintained]: bitmaps is unmaintained
ID: RUSTSEC-2026-0247
Solution: No safe upgrade is available!
`bitmaps` is transitive only, via matrix-sdk -> imbl ->
imbl-sized-chunks. It is an unmaintained notice, not a vulnerability,
and cargo-deny itself reports that no safe upgrade exists. That is
exactly the class `[advisories].ignore` already documents (bincode via
syntect, paste via fastembed, proc-macro-error2 via Matrix/Teloxide,
rustls-pemfile via qdrant-client), so it is ignored on the same terms.
Vulnerability advisories are still upgraded rather than ignored — see
(2).
2. Patch the two crates behind all five open Dependabot alerts. Both are
reachable from the workspace and both have released fixes:
- russh 0.62.2 -> 0.62.5 (direct dep of crates/terminal), closing
GHSA-m65r-rprj-r5rg (CVE-2026-68930, channel-scoped server
callbacks reachable without an open channel), GHSA-g9hv-x236-4qp3
(pre-auth X25519 wrong-length panic), GHSA-cqjc-rmpq-xprq
(post-auth pty-req panic) and GHSA-5xvq-cp9x-6p6r (pre-auth
all-zero Curve25519 encode_mpint OOB).
- quinn-proto 0.11.14 -> 0.11.16 (transitive), closing the one HIGH
alert, GHSA-4w2j-m93h-cj5j (remote memory exhaustion from
unbounded out-of-order stream reassembly).
Dependabot did raise these as #385 and #382, but against `main`, where
they fail DCO; `dev` never received them.
3. Resync Cargo.lock, which had drifted from the manifests: it was
missing `ardur-delegate-tool` (#332) and `ardur-durability` (#327)
along with tar/filetime/xattr, and still carried stale entries.
`.github/workflows/release.yml` builds `--locked`, so a release cut
from `dev` would have failed to resolve. `cargo metadata --locked` now
succeeds.
Verified locally: `cargo deny check` reports advisories/bans/licenses/
sources all ok; `cargo fmt --check`, `cargo build --workspace` and
`cargo clippy --workspace --all-targets -D warnings` are clean.
Checkpoint: architect/sessions/merge-audit-lane-2026-08-11/journal.md
Signed-off-by: GR <gnanirn@gmail.com>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Three coupled fixes that together restore a mergeable
dev.cargo-deny (required check) is failing on every open PR and on
devitself with a single finding:
bitmapsis transitive only, via matrix-sdk -> imbl ->imbl-sized-chunks. It is an unmaintained notice, not a vulnerability,
and cargo-deny itself reports that no safe upgrade exists. That is
exactly the class
[advisories].ignorealready documents (bincode viasyntect, paste via fastembed, proc-macro-error2 via Matrix/Teloxide,
rustls-pemfile via qdrant-client), so it is ignored on the same terms.
Vulnerability advisories are still upgraded rather than ignored — see
(2).
Patch the two crates behind all five open Dependabot alerts. Both are
reachable from the workspace and both have released fixes:
GHSA-m65r-rprj-r5rg (CVE-2026-68930, channel-scoped server
callbacks reachable without an open channel), GHSA-g9hv-x236-4qp3
(pre-auth X25519 wrong-length panic), GHSA-cqjc-rmpq-xprq
(post-auth pty-req panic) and GHSA-5xvq-cp9x-6p6r (pre-auth
all-zero Curve25519 encode_mpint OOB).
alert, GHSA-4w2j-m93h-cj5j (remote memory exhaustion from
unbounded out-of-order stream reassembly).
Dependabot did raise these as chore(deps): Bump russh from 0.62.1 to 0.62.5 #385 and chore(deps): Bump quinn-proto from 0.11.14 to 0.11.16 #382, but against
main, wherethey fail DCO;
devnever received them.Resync Cargo.lock, which had drifted from the manifests: it was
missing
ardur-delegate-tool(feat(§5.1): delegate_task tool over the ardur-multi-agent substrate #332) andardur-durability(feat(durability): atomic writes, backup/restore, and schema versioning for durable state #327)along with tar/filetime/xattr, and still carried stale entries.
.github/workflows/release.ymlbuilds--locked, so a release cutfrom
devwould have failed to resolve.cargo metadata --lockednowsucceeds.
Verified locally:
cargo deny checkreports advisories/bans/licenses/sources all ok;
cargo fmt --check,cargo build --workspaceandcargo clippy --workspace --all-targets -D warningsare clean.Why this lands first:
cargo-denyis a required check and is currently red on every open PR (#368, #369, #370, #372, #373, #374, #375 all show exactly one failing required check, and it is this one). Nothing can merge intodevuntil the advisory is resolved.