Repository navigation
deps: fix Rust and CI pytest security advisories - #140
Merged
Merged
Conversation
puretensor
added a commit
that referenced
this pull request
Oct 19, 2026
… codex gpt-6.1-sol) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TNEBGCM67tZXYzHdTUPkPs
… codex gpt-6.1-sol) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TNEBGCM67tZXYzHdTUPkPs
puretensor
force-pushed
the
codex/upgrade-ptask-language-dependencies
branch
from
October 19, 2026 15:27
f5d736d to
0f10513
Compare
… codex gpt-6.1-sol) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TNEBGCM67tZXYzHdTUPkPs
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.
Fix five quinn-proto denial-of-service advisories, the memmap2 and lru unsoundness advisories, and pytest's insecure temporary-directory advisory. The production application and existing tests require no code changes.
BEFORE / AFTER audits
Severity is taken from the advisory CVSS vector / GitHub advisory; Unscored means no severity is published, rather than an inferred rating. pip-audit emitted the same pytest advisory twice; counts distinguish raw records from unique IDs.
Every version change
New transitive lock entries required by these upgrades: approx 0.5.1, by_address 1.2.1, critical-section 1.2.0, palette 0.7.7, palette_derive 0.7.7, palette_math 0.7.7, rand_pcg 0.10.2, strum 0.28.0, strum_macros 0.28.0. The strum 0.28 entries coexist with older locked strum versions. No container digest changes.
Compatibility and scope
Updates use Cargo's precise-version resolver; lockfile hashes are generated by Cargo. The smallest public dependency surface changes are two patch releases and ratatui-core 0.1.2, whose updated lru constraint permits the fixed 0.18 line. lru has no fix in its old 0.16 compatibility line; the compatible parent patch absorbs its API change. bitflags and the new lock entries are required transitive changes. No application code, test assertions, package version, or deny policy changes.
pytest has no fixed 8.x release: 9.0.3 is the latest 9.0 patch and first fixed release. Reviewed the pytest changelog, including the 9.0 breaking-change list (Python >=3.10; overlapping/duplicate collection arguments; deprecated APIs now error). CI uses Python 3.x and two distinct test files; these tests need no workaround and all 51 pass unchanged. The Python major bump is limited to CI test tooling.
Remaining findings / HOLDS
Validation
The clean-main baseline at
715d04b6c32ac93c31dd112df05617bcbccd6972passed the requested gate andcargo deny check advisories licenses sources. No pre-existing failures.On this branch, with
CARGO_TARGET_DIR=$PWD/target,CARGO_BUILD_JOBS=16, installed Rust 1.98.1 (matching CI), and a temporary test HOME:cargo fmt --all -- --check && cargo clippy --workspace --all-targets -- -D warnings && cargo test --workspace --locked— exit 0; 894 passed, 0 failed, 0 ignored across Rust test suites.cargo deny check advisories licenses sources— exit 0:advisories ok, licenses ok, sources ok(existing all-features graph policy; metadata only, no native-ml compilation).uvx pip-audit -r <extracted CI pin>— AFTER:No known vulnerabilities found.51 passed in 26.78s.actionlint— exit 0.cargo audit --json— AFTER: 3 vulnerabilities and 2 unmaintained notices; exits 1 for the explicitly held zip advisories. All seven claimed Rust advisory removals were checked by ID against BEFORE/AFTER JSON.Gate final output:
No native-ml feature build, deployment, container build, production datastore tests, or external messaging tooling was invoked. Tests use temporary SQLite databases and local fake services.