Repository navigation
Build(deps): Bump zstd from 0.13.4 to 0.14.3 in /praxis - #1755
Conversation
Bumps [zstd](https://github.com/gyscos/zstd-rs) from 0.13.4 to 0.14.3. - [Release notes](https://github.com/gyscos/zstd-rs/releases) - [Commits](gyscos/zstd-rs@v0.13.4...v0.14.3) --- updated-dependencies: - dependency-name: zstd dependency-version: 0.14.3 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
|
Important Review skippedBot user detected. To trigger a single review, invoke the
|
clawgenti
left a comment
There was a problem hiding this comment.
Summary
This is a routine Dependabot bump of the zstd crate from 0.13.4 to 0.14.3 (and zstd-safe 7.4.0 to 8.2.0) in the praxis Rust workspace. The diff is minimal: one requirement line in praxis/filters/Cargo.toml and two package entries in praxis/Cargo.lock. Both checksums match crates.io. I built and tested the PR head locally: cargo test --locked passes (53 unit + 4 parity tests) and cargo clippy --locked --all-targets -- -D warnings is clean on the pinned 1.96.0 toolchain. No must-fix issues found.
Author: dependabot[bot] (CONTRIBUTOR, trusted automated dependency bot)
Areas reviewed: Rust dependency manifests (Cargo.toml/Cargo.lock), API usage in praxis/filters/src/parse/decode.rs, supply chain
Agent/IDE config (.claude/.vscode): none
Commits: 1 commit, all signed-off: yes (DCO pass)
CI status: pending. At review time only DCO has passed. The other 37 checks are queued or running, 0 have failed, and 2 are skipped (Spellcheck, CodeRabbit). The Rust (praxis) job is still queued, so please confirm it and Dependency Review go green before merging.
Upstream / transitive check
- Checksums:
zstd 0.14.3(db3ae9bf…3c70) andzstd-safe 8.2.0(ded520f1…434b) match the crates.io index. Neither is yanked.zstd-sysstays at2.1.2+zstd.1.5.7, which satisfies 0.14.3's^2.1.2requirement, so the lockfile delta is limited to these two packages, as expected. - MSRV: both crates declare
rust-version = "1.64". praxis pins 1.96.0 (rust-version = "1.96"), so that's fine. - Breaking changes in 0.14.0:
with_prepared_dictionary()constructors now borrow the dictionary for the stream's lifetime. praxis doesn't use dictionaries.zstd-safe8 adds a'alifetime onref_cdict/ref_ddict. Not used here.read::Decoder::finish()now consumes the rest of the frame. praxis never callsfinish()on the read decoder.
- Behavioral fixes worth knowing about. Both are compatible with how praxis uses the crate:
zstd-safenow commitsInBuffer/OutBufferpositions only after a successful streaming call (006e1d1). The streaming path indecode.rsmaps anyrun()error toDecodeError::Corruptand treats it as final, so it never reads positions after a failure.read_to_endbuffer growth is reduced (f302406). That affectsdecode_request_body, but the 64 MiBtake()cap and its semantics are unchanged.
- API calls in praxis checked against 0.14.3. Signatures are unchanged for
raw::Decoder::new(),set_parameter(DParameter::WindowLogMax),Operation::runwithInBuffer::around/OutBuffer::around,read::Decoder::new+window_log_max,encode_all, and theEncoderbuilders used in tests (window_log,long_distance_matching,include_contentsize). It compiles and thezstd_window_limitstest (RFC 9659 8 MiB window / 2^26 request window) passes. - License: upstream relicensed from MIT to BSD-3-Clause in 0.14.0. That's permissive and not in the repo's
deny-licenses: GPL-3.0, AGPL-3.0list, so it's fine. Mentioning it for anyone who tracks third-party license notices.
Bumps zstd from 0.13.4 to 0.14.3.
Commits
75bc740Release zstd 0.14.3, zstd-safe 8.2.0, and zstd-sys 2.1.21c580b6zstd-safe: keep the one-shot reset check inline502ee2fzstd-safe: only reset after one-shot decompression if a stream may be open5ded6e0zstd-safe: apply rustfmte224018zstd-safe: refuse to unload a dictionary under unfinished zstdmt jobs6a1065bStop a single_frame() decoder when finish_frame() ends the framec7156dfDon't let finish_frame() start a frame nothing was read fromb0fb8a7Add write::Decoder::finish() to report incomplete framesab34d01Don't retry a panicked writer when dropping AutoFlushDecoder/AutoFinishEncoder74ace2dReset the borrowed context's session in raw::{Encoder,Decoder}::with_contextDependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)