Repository navigation
build(deps): bump cachekit-core to 0.7.2 (LAB-11012) - #38
Merged
Merged
Conversation
Lockfile only: Cargo.toml's "0.7" already admits 0.7.2. The bump also drops twox-hash, which core 0.7.2 no longer pulls in through lz4_flex's frame feature.
Contributor
Author
|
@coderabbitai review |
Contributor
Author
|
@kody start-review |
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.
What
Moves
Cargo.lockfrom cachekit-core 0.7.1 to 0.7.2 withcargo update -p cachekit-core --precise 0.7.2.Cargo.tomlis unchanged:"0.7"already admits 0.7.2, and so does cachekit-rs 0.11.1's requirement.Lock entries that moved:
cachekit-core9bc8d8f4…)10167217…)lz4_flex0.12.2twox-hashframefeature)twox-hash2.1.4Nothing else moved. One cachekit-core resolves, for both the direct dependency and cachekit-rs (
cargo tree -i cachekit-core --target all), whichdeny.toml'sdeny-multiple-versionsalso enforces.Effect on
ckNone on what
ckruns or writes.ckstores through cachekit-rs'sEncryptionLayer, which seals the plaintext with core's nativeencrypt_aes_gcmand never goes throughByteStorageor LZ4. 0.7.2's changes are LZ4-side (the native encoder, the literals-only block below 64 B) and a wasm32-only AES-GCM seal; nativeencrypt_aes_gcmandderive_domain_keyare unchanged.ckdoes store payloads under 64 B (the 16-byte failure marker, and an envelope of 8 bytes plus short stdout), but they are sealed uncompressed, so the sub-64 B change does not reach them.what_ck_writes_the_frozen_format_readspasses on 0.7.2 and pins that: every size it writes, from a 9-byte envelope up, opens as plainnonce ‖ AES-GCM(envelope).This repo has no instruction-count harness, so there is no A/B table here. The paths
ckuses are the ones cachekit-rs measures.Checks (local, as CI runs them)
cargo fmt --all -- --check: passcargo clippy --locked --all-targets -- -D warnings: passcargo deny check: advisories ok, bans ok, licenses ok, sources okcargo metadata --locked: pass; no path dependencies outside the workspacecargo test --locked: lib 22 passed, readme 1, run 45, saas 17; bench 1 ignored (run below); 0 failedcargo build --locked --release, then the admin-API greps on the source and the binary: no matchcargo test --locked --release --test bench -- --ignored: hit-path budget passescargo +1.91 check --locked --all-targets(rust-version 1.91, the msrv job): pass. cachekit-core 0.7.2 declaresrust-version = "1.91", matching this crate's.No docs change: README and the docs here cite no cachekit-core version, compression behaviour or benchmark numbers.
Closes LAB-11012